[jboss-cvs] JBossAS SVN: r64563 - trunk/tomcat/src/main/org/jboss/web/tomcat/security.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Mon Aug 13 15:23:11 EDT 2007


Author: anil.saldhana at jboss.com
Date: 2007-08-13 15:23:10 -0400 (Mon, 13 Aug 2007)
New Revision: 64563

Modified:
   trunk/tomcat/src/main/org/jboss/web/tomcat/security/JBossWebRealm.java
Log:
remove the unnecessary check

Modified: trunk/tomcat/src/main/org/jboss/web/tomcat/security/JBossWebRealm.java
===================================================================
--- trunk/tomcat/src/main/org/jboss/web/tomcat/security/JBossWebRealm.java	2007-08-13 19:22:56 UTC (rev 64562)
+++ trunk/tomcat/src/main/org/jboss/web/tomcat/security/JBossWebRealm.java	2007-08-13 19:23:10 UTC (rev 64563)
@@ -469,9 +469,7 @@
                    super.hasResourcePermission(request,response, 
                                       securityConstraints, context);  
       
-      Subject caller = this.establishSubjectContext(request.getPrincipal());
-      if(caller == null)
-         throw new IllegalStateException("Caller subject is null");
+      Subject caller = this.establishSubjectContext(request.getPrincipal()); 
 
       Map<String,Object> map =  new HashMap<String,Object>(); 
       map.put(ResourceKeys.WEB_REQUEST, request);




More information about the jboss-cvs-commits mailing list