[jboss-cvs] JBossAS SVN: r57955 - projects/security/trunk/src/main/org/jboss/security/plugins

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

Author: anil.saldhana at jboss.com
Date: 2006-10-31 16:46:32 -0500 (Tue, 31 Oct 2006)
New Revision: 57955

Modified:
   projects/security/trunk/src/main/org/jboss/security/plugins/JBossAuthorizationManager.java
Log:
Change in mapping context api

Modified: projects/security/trunk/src/main/org/jboss/security/plugins/JBossAuthorizationManager.java
===================================================================
--- projects/security/trunk/src/main/org/jboss/security/plugins/JBossAuthorizationManager.java	2006-10-31 21:45:59 UTC (rev 57954)
+++ projects/security/trunk/src/main/org/jboss/security/plugins/JBossAuthorizationManager.java	2006-10-31 21:46:32 UTC (rev 57955)
@@ -329,7 +329,7 @@
             contextMap.put(SecurityConstants.PRINCIPALS_IDENTIFIER, subject.getPrincipals());
             if(trace)
                log.trace("Roles before mapping:"+ userRoles);
-            userRoles = (Group)mc.performMapping(contextMap);
+            mc.performMapping(contextMap, userRoles);
             if(trace)
                log.trace("Roles after mapping:"+ userRoles);
          } 
@@ -362,6 +362,9 @@
       return source;
    }
 
+   /**
+    * @see AuthorizationManager#getTargetRoles(Principal, Map)
+    */
    public Group getTargetRoles(Principal targetPrincipal, Map contextMap)
    {
       throw new RuntimeException("Not implemented");