[jboss-cvs] JBossAS SVN: r110378 - branches/JBPAPP_4_2_0_GA_CP/security/src/main/org/jboss/security/auth/spi.

[jboss-cvs-commits at lists.jboss.org]

Author: mmoyses
Date: 2011-01-14 11:53:02 -0500 (Fri, 14 Jan 2011)
New Revision: 110378

Modified:
   branches/JBPAPP_4_2_0_GA_CP/security/src/main/org/jboss/security/auth/spi/RoleMappingLoginModule.java
Log:
fixing NPE

Modified: branches/JBPAPP_4_2_0_GA_CP/security/src/main/org/jboss/security/auth/spi/RoleMappingLoginModule.java
===================================================================
--- branches/JBPAPP_4_2_0_GA_CP/security/src/main/org/jboss/security/auth/spi/RoleMappingLoginModule.java	2011-01-14 16:50:41 UTC (rev 110377)
+++ branches/JBPAPP_4_2_0_GA_CP/security/src/main/org/jboss/security/auth/spi/RoleMappingLoginModule.java	2011-01-14 16:53:02 UTC (rev 110378)
@@ -178,10 +178,13 @@
          String roleKey = (String)enumer.nextElement();
          String comma_separated_roles = props.getProperty(roleKey);
          Principal pIdentity = createIdentity(roleKey);
-         if(group.isMember(pIdentity))
-            Util.parseGroupMembers(group,comma_separated_roles,this);
-         if(REPLACE_ROLE)
-            group.removeMember(pIdentity); 
+         if (group != null)
+         {
+            if(group.isMember(pIdentity))
+               Util.parseGroupMembers(group,comma_separated_roles,this);
+            if(REPLACE_ROLE)
+               group.removeMember(pIdentity);
+         }
       } 
    }
 }