[jboss-cvs] JBossAS SVN: r80357 - projects/security/security-jboss-sx/trunk/jbosssx/src/main/java/org/jboss/security.

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

Author: anil.saldhana at jboss.com
Date: 2008-11-02 18:51:56 -0500 (Sun, 02 Nov 2008)
New Revision: 80357

Modified:
   projects/security/security-jboss-sx/trunk/jbosssx/src/main/java/org/jboss/security/SecurityRolesAssociation.java
Log:
SECURITY-306: use remove() instead of set(null) on ThreadLocal

Modified: projects/security/security-jboss-sx/trunk/jbosssx/src/main/java/org/jboss/security/SecurityRolesAssociation.java
===================================================================
--- projects/security/security-jboss-sx/trunk/jbosssx/src/main/java/org/jboss/security/SecurityRolesAssociation.java	2008-11-02 23:49:33 UTC (rev 80356)
+++ projects/security/security-jboss-sx/trunk/jbosssx/src/main/java/org/jboss/security/SecurityRolesAssociation.java	2008-11-02 23:51:56 UTC (rev 80357)
@@ -61,6 +61,9 @@
    {
       if(log.isTraceEnabled())
          log.trace("Setting threadlocal:"+securityRoles);
-      threadSecurityRoleMapping.set(securityRoles);
+      if(securityRoles == null)
+         threadSecurityRoleMapping.remove();
+      else
+         threadSecurityRoleMapping.set(securityRoles);
    }
 }
\ No newline at end of file