You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@river.apache.org by pe...@apache.org on 2010/08/14 13:13:51 UTC

svn commit: r985461 - in /incubator/river/jtsk/trunk/src/org/apache/river: api/security/ExecutionContextManager.java imp/security/policy/se/ECM.java

Author: peter_firmstone
Date: Sat Aug 14 11:13:51 2010
New Revision: 985461

URL: http://svn.apache.org/viewvc?rev=985461&view=rev
Log:
ExecutionContextManager implementation for the RevokableDynamicPolicy, please review.

Modified:
    incubator/river/jtsk/trunk/src/org/apache/river/api/security/ExecutionContextManager.java
    incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/se/ECM.java

Modified: incubator/river/jtsk/trunk/src/org/apache/river/api/security/ExecutionContextManager.java
URL: http://svn.apache.org/viewvc/incubator/river/jtsk/trunk/src/org/apache/river/api/security/ExecutionContextManager.java?rev=985461&r1=985460&r2=985461&view=diff
==============================================================================
--- incubator/river/jtsk/trunk/src/org/apache/river/api/security/ExecutionContextManager.java (original)
+++ incubator/river/jtsk/trunk/src/org/apache/river/api/security/ExecutionContextManager.java Sat Aug 14 11:13:51 2010
@@ -138,8 +138,10 @@ public interface ExecutionContextManager
      * 
      * @param p Permission to be checked, if result not already in cache.
      * @throws java.security.AccessControlException
+     * @throws java.lang.NullPointerException 
      */
-    public void checkPermission(Permission p) throws AccessControlException;
+    public void checkPermission(Permission p) throws AccessControlException,
+	    NullPointerException;
     
     /**
      * <p>

Modified: incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/se/ECM.java
URL: http://svn.apache.org/viewvc/incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/se/ECM.java?rev=985461&r1=985460&r2=985461&view=diff
==============================================================================
--- incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/se/ECM.java (original)
+++ incubator/river/jtsk/trunk/src/org/apache/river/imp/security/policy/se/ECM.java Sat Aug 14 11:13:51 2010
@@ -196,6 +196,7 @@ class ECM implements ExecutionContextMan
     }
 
     public void checkPermission(Permission p) throws AccessControlException {
+	if (p == null ) throw new NullPointerException("Permission null");
 	Thread currentThread = Thread.currentThread();
 	AccessControlContext executionContext = AccessController.getContext();
 	rl.lock();