You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-dev@db.apache.org by "Kristian Waagan (JIRA)" <ji...@apache.org> on 2008/10/07 09:41:44 UTC
[jira] Updated: (DERBY-3728) Fix error handling in
PrivilegedFileOpsForTests
[ https://issues.apache.org/jira/browse/DERBY-3728?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Waagan updated DERBY-3728:
-----------------------------------
Derby Info: [Patch Available]
> Fix error handling in PrivilegedFileOpsForTests
> -----------------------------------------------
>
> Key: DERBY-3728
> URL: https://issues.apache.org/jira/browse/DERBY-3728
> Project: Derby
> Issue Type: Bug
> Components: Test
> Affects Versions: 10.5.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Priority: Minor
> Fix For: 10.5.0.0
>
> Attachments: derby-3728-1a.diff
>
>
> The exception handling in this utility class is unclear and can in some cases cause ClassCastException to be thrown.
> Note that SecurityException is a runtime exception.
> Potential issues to be fixed:
> 1) Avoid possibility of ClassCastException in 'getFileInputStream'
> 2) Don't include SecurityException in the signature (throws clause)
> 3) Document SecurityException in JavaDoc.
> Points 2 and 3 leads to the option of not catching SecurityException, which allows the code to be written more concisely and we can use PrivilegedAction instead of PrivilegedExceptionAction in some cases.
> There is also a missing character in the class name in the header, and an invalid JavaDoc tag in the class JavaDoc.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.