You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by jd...@apache.org on 2008/06/18 18:55:47 UTC

svn commit: r669219 - /geronimo/gshell/trunk/gshell-plugin/src/main/java/org/apache/geronimo/gshell/application/ApplicationSecurityManager.java

Author: jdillon
Date: Wed Jun 18 09:55:47 2008
New Revision: 669219

URL: http://svn.apache.org/viewvc?rev=669219&view=rev
Log:
Disable checks for now, they are broken

Modified:
    geronimo/gshell/trunk/gshell-plugin/src/main/java/org/apache/geronimo/gshell/application/ApplicationSecurityManager.java

Modified: geronimo/gshell/trunk/gshell-plugin/src/main/java/org/apache/geronimo/gshell/application/ApplicationSecurityManager.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-plugin/src/main/java/org/apache/geronimo/gshell/application/ApplicationSecurityManager.java?rev=669219&r1=669218&r2=669219&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-plugin/src/main/java/org/apache/geronimo/gshell/application/ApplicationSecurityManager.java (original)
+++ geronimo/gshell/trunk/gshell-plugin/src/main/java/org/apache/geronimo/gshell/application/ApplicationSecurityManager.java Wed Jun 18 09:55:47 2008
@@ -57,6 +57,11 @@
         // TODO: See if there is a more efficent and/or recommended way to implement custom permission handling
         //
 
+        //
+        // FIXME: These don't work as desired ATM, so disable and re-implement a little bit later
+        //
+        
+        /*
         if (perm instanceof RuntimePermission) {
             // Prevent System.exit()
             if (perm.implies(new RuntimePermission("exitVM"))) {
@@ -75,6 +80,7 @@
                 throw new SecurityException();
             }
         }
+        */
         
         if (parent != null) {
             parent.checkPermission(perm);