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;
}