[jboss-cvs] Picketbox SVN: r157 - trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi.
jboss-cvs-commits at lists.jboss.org
jboss-cvs-commits at lists.jboss.org
Fri Jan 14 11:50:29 EST 2011
Author: mmoyses
Date: 2011-01-14 11:50:28 -0500 (Fri, 14 Jan 2011)
New Revision: 157
Modified:
trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi/RoleMappingLoginModule.java
Log:
SECURITY-554: fixing NPE
Modified: trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi/RoleMappingLoginModule.java
===================================================================
--- trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi/RoleMappingLoginModule.java 2011-01-14 13:29:20 UTC (rev 156)
+++ trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi/RoleMappingLoginModule.java 2011-01-14 16:50:28 UTC (rev 157)
@@ -165,10 +165,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);
+ }
}
}
-}
\ No newline at end of file
+}
More information about the jboss-cvs-commits
mailing list