db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Knut Anders Hatlen (JIRA)" <j...@apache.org>
Subject [jira] Commented: (DERBY-2564) ContextService.notifyAllActiveThreads() needs a privileged block around the call to Thread.interrupt()
Date Fri, 16 Nov 2007 19:12:43 GMT

    [ https://issues.apache.org/jira/browse/DERBY-2564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12543172
] 

Knut Anders Hatlen commented on DERBY-2564:
-------------------------------------------

The patch looks good! Letting the exception propagate out to the caller is the correct thing
to do, so that should be fine. Since SecurityException is a RuntimeException, you could even
skip the "throws SecurityException" part. Two of the imports are unused, by the way (Policy
and PrivilegedActionException). Other than that, +1 to commit.

> ContextService.notifyAllActiveThreads() needs a privileged block around the call to Thread.interrupt()
> ------------------------------------------------------------------------------------------------------
>
>                 Key: DERBY-2564
>                 URL: https://issues.apache.org/jira/browse/DERBY-2564
>             Project: Derby
>          Issue Type: Bug
>          Components: Security
>    Affects Versions: 10.0.2.0, 10.0.2.1, 10.1.1.0, 10.1.2.1, 10.1.3.1, 10.2.1.6, 10.2.2.0
>            Reporter: Rick Hillegas
>            Assignee: Kathey Marsden
>         Attachments: derby-2564_diff.txt, derby-2564_stat.txt
>
>
> It looks to me as though the call to interrupt() can raise a SecurityException. I think
this may give rise to another permission that we need to add to our template policy file and
to our security documentation.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message