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

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

Author: anil.saldhana at jboss.com
Date: 2008-11-02 18:49:33 -0500 (Sun, 02 Nov 2008)
New Revision: 80356

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

Modified: projects/security/security-spi/trunk/spi/src/main/org/jboss/security/SecurityContextAssociation.java
===================================================================
--- projects/security/security-spi/trunk/spi/src/main/org/jboss/security/SecurityContextAssociation.java	2008-11-02 22:52:19 UTC (rev 80355)
+++ projects/security/security-spi/trunk/spi/src/main/org/jboss/security/SecurityContextAssociation.java	2008-11-02 23:49:33 UTC (rev 80356)
@@ -91,7 +91,12 @@
       if(!SERVER)
          securityContext = sc;
       else
-         securityContextLocal.set(sc);
+      {
+         if(sc == null)
+            securityContextLocal.remove();
+         else
+            securityContextLocal.set(sc); 
+      }
    }
    
    public static SecurityContext getSecurityContext()
@@ -107,7 +112,7 @@
       if(!SERVER)
          securityContext = null;
       else
-         securityContextLocal.set(null);
+         securityContextLocal.remove();
    }