You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by tr...@apache.org on 2007/04/18 09:31:59 UTC

svn commit: r529899 - /mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java

Author: trustin
Date: Wed Apr 18 00:31:57 2007
New Revision: 529899

URL: http://svn.apache.org/viewvc?view=rev&rev=529899
Log:
Removed the possibility of returning different locks without a synchronized block in ProtocolCodecFilter

Modified:
    mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java

Modified: mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java?view=diff&rev=529899&r1=529898&r2=529899
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java (original)
+++ mina/trunk/core/src/main/java/org/apache/mina/filter/codec/ProtocolCodecFilter.java Wed Apr 18 00:31:57 2007
@@ -330,8 +330,7 @@
         Object lock = session.getAttribute( DECODER_LOCK );
         if( lock == null )
         {
-            lock = new Object();
-            session.setAttribute( DECODER_LOCK, lock );
+            lock = session.setAttributeIfAbsent( DECODER_LOCK, new Object() );
         }
 
         return lock;