You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by tr...@apache.org on 2006/03/12 05:52:02 UTC

svn commit: r385245 - /directory/trunks/mina/core/src/main/java/org/apache/mina/common/support/BaseIoSession.java

Author: trustin
Date: Sat Mar 11 20:52:01 2006
New Revision: 385245

URL: http://svn.apache.org/viewcvs?rev=385245&view=rev
Log:
Related issue: DIRMINA-181 (Failure to shutdown SSL in a test run)
* Fixed: BaseIoSession.isClosing() was implemented incorrectly.
* Fixed: BaseIoSession.close() didn't prevent duplicate invocations.

Modified:
    directory/trunks/mina/core/src/main/java/org/apache/mina/common/support/BaseIoSession.java

Modified: directory/trunks/mina/core/src/main/java/org/apache/mina/common/support/BaseIoSession.java
URL: http://svn.apache.org/viewcvs/directory/trunks/mina/core/src/main/java/org/apache/mina/common/support/BaseIoSession.java?rev=385245&r1=385244&r2=385245&view=diff
==============================================================================
--- directory/trunks/mina/core/src/main/java/org/apache/mina/common/support/BaseIoSession.java (original)
+++ directory/trunks/mina/core/src/main/java/org/apache/mina/common/support/BaseIoSession.java Sat Mar 11 20:52:01 2006
@@ -86,7 +86,7 @@
     
     public synchronized boolean isClosing()
     {
-        return closing;
+        return closing || closeFuture.isClosed();
     }
     
     public CloseFuture getCloseFuture()
@@ -98,7 +98,11 @@
     {
         synchronized( this )
         {
-            if( !closing )
+            if( isClosing() )
+            {
+                return closeFuture;
+            }
+            else
             {
                 closing = true;
             }