You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@felix.apache.org by Herko ter Horst <he...@terhorst.net> on 2006/03/05 20:17:53 UTC
Bug (+fix) in SecureAction
Hi,
I ran into this issue (a copy/paste mistake) while trying to run Felix
from WebStart. The issue doesn't seem to occur when running normally.
Patch is below.
Cheers,
Herko
Index: src/main/java/org/apache/felix/framework/util/SecureAction.java
===================================================================
--- src/main/java/org/apache/felix/framework/util/SecureAction.java
(revision 383309)
+++ src/main/java/org/apache/felix/framework/util/SecureAction.java
(working copy)
@@ -208,7 +208,7 @@
{
try
{
- m_actions.set(Actions.MAKE_DIRECTORY_ACTION, file);
+ m_actions.set(Actions.LIST_DIRECTORY_ACTION, file);
return (File[])
AccessController.doPrivileged(m_actions, m_acc);
}
catch (PrivilegedActionException ex)
Re: Bug (+fix) in SecureAction
Posted by "Richard S. Hall" <he...@ungoverned.org>.
Thanks a lot for catching that...your fix is now committed.
-> richard
Herko ter Horst wrote:
> Hi,
>
> I ran into this issue (a copy/paste mistake) while trying to run Felix
> from WebStart. The issue doesn't seem to occur when running normally.
> Patch is below.
>
> Cheers,
>
> Herko
>
>
> Index: src/main/java/org/apache/felix/framework/util/SecureAction.java
> ===================================================================
> --- src/main/java/org/apache/felix/framework/util/SecureAction.java
> (revision 383309)
> +++ src/main/java/org/apache/felix/framework/util/SecureAction.java
> (working copy)
> @@ -208,7 +208,7 @@
> {
> try
> {
> - m_actions.set(Actions.MAKE_DIRECTORY_ACTION, file);
> + m_actions.set(Actions.LIST_DIRECTORY_ACTION, file);
> return (File[])
> AccessController.doPrivileged(m_actions, m_acc);
> }
> catch (PrivilegedActionException ex)
>