[jboss-cvs] JBossAS SVN: r62297 - trunk/jbossmq/src/main/org/jboss/mq/sm.

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

Author: adrian at jboss.org
Date: 2007-04-12 06:51:48 -0400 (Thu, 12 Apr 2007)
New Revision: 62297

Modified:
   trunk/jbossmq/src/main/org/jboss/mq/sm/AbstractStateManager.java
Log:
[JBAS-4324] - Fix race condition in client id checking

Modified: trunk/jbossmq/src/main/org/jboss/mq/sm/AbstractStateManager.java
===================================================================
--- trunk/jbossmq/src/main/org/jboss/mq/sm/AbstractStateManager.java	2007-04-12 10:38:49 UTC (rev 62296)
+++ trunk/jbossmq/src/main/org/jboss/mq/sm/AbstractStateManager.java	2007-04-12 10:51:48 UTC (rev 62297)
@@ -190,16 +190,12 @@
 
    public void addLoggedOnClientId(String ID) throws JMSException
    {
-      synchronized (loggedOnClientIds)
-      {
-         if (loggedOnClientIds.contains(ID))
-            throw new InvalidClientIDException("This client id '" + ID + "' is already registered!");
-      }
-      
       checkLoggedOnClientId(ID);
 
       synchronized (loggedOnClientIds)
       {
+         if (loggedOnClientIds.contains(ID))
+            throw new InvalidClientIDException("This client id '" + ID + "' is already registered!");
          loggedOnClientIds.add(ID);
       }
       if (log.isTraceEnabled())
@@ -384,4 +380,4 @@
          return buffer.toString();
       }
    }
-}
\ No newline at end of file
+}