[jboss-cvs] JBossAS SVN: r58659 - projects/security/trunk/src/main/org/jboss/security/auth/spi

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

Author: scott.stark at jboss.org
Date: 2006-11-21 16:07:53 -0500 (Tue, 21 Nov 2006)
New Revision: 58659

Modified:
   projects/security/trunk/src/main/org/jboss/security/auth/spi/RoleMappingLoginModule.java
Log:
Remove the check for a SimpleGroup as the standard Group interface is all that is needed.

Modified: projects/security/trunk/src/main/org/jboss/security/auth/spi/RoleMappingLoginModule.java
===================================================================
--- projects/security/trunk/src/main/org/jboss/security/auth/spi/RoleMappingLoginModule.java	2006-11-21 20:58:07 UTC (rev 58658)
+++ projects/security/trunk/src/main/org/jboss/security/auth/spi/RoleMappingLoginModule.java	2006-11-21 21:07:53 UTC (rev 58659)
@@ -33,7 +33,6 @@
 import javax.security.auth.login.LoginException;
 
 import org.jboss.logging.Logger;
-import org.jboss.security.SimpleGroup;
 import org.jboss.util.StringPropertyReplacer;
 
 //$Id$
@@ -154,16 +153,16 @@
       while(iter.hasNext())
       {
          Principal p = (Principal)iter.next();
-         if(p instanceof SimpleGroup)
+         if(p instanceof Group)
          {
-           SimpleGroup sg = (SimpleGroup)p;
-           if("Roles".equals(sg.getName()))
-              return sg;
+           Group g = (Group) p;
+           if("Roles".equals(g.getName()))
+              return g;
          } 
       }
       return null;
    }
-   
+
    /**
     * Process the group with the roles that are mapped in the 
     * properies file