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/10/31 08:34:53 UTC

svn commit: r590580 - in /mina/trunk/core/src/main/java/org/apache/mina/common: AbstractIoSession.java DefaultIoFilterChain.java

Author: trustin
Date: Wed Oct 31 00:34:52 2007
New Revision: 590580

URL: http://svn.apache.org/viewvc?rev=590580&view=rev
Log:
Forgot to call attributeMap.dispose()

Modified:
    mina/trunk/core/src/main/java/org/apache/mina/common/AbstractIoSession.java
    mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChain.java

Modified: mina/trunk/core/src/main/java/org/apache/mina/common/AbstractIoSession.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/common/AbstractIoSession.java?rev=590580&r1=590579&r2=590580&view=diff
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/common/AbstractIoSession.java (original)
+++ mina/trunk/core/src/main/java/org/apache/mina/common/AbstractIoSession.java Wed Oct 31 00:34:52 2007
@@ -302,6 +302,10 @@
         return attributes.getAttributeKeys(this);
     }
     
+    protected IoSessionAttributeMap getAttributeMap() {
+        return attributes;
+    }
+
     protected void setAttributeMap(IoSessionAttributeMap attributes) {
         this.attributes = attributes;
     }

Modified: mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChain.java
URL: http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChain.java?rev=590580&r1=590579&r2=590580&view=diff
==============================================================================
--- mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChain.java (original)
+++ mina/trunk/core/src/main/java/org/apache/mina/common/DefaultIoFilterChain.java Wed Oct 31 00:34:52 2007
@@ -670,8 +670,12 @@
             try {
                 session.getHandler().sessionClosed(session);
             } finally {
-                // Remove all filters.
-                session.getFilterChain().clear();
+                try {
+                    ((AbstractIoSession) session).getAttributeMap().dispose(session);
+                } finally {
+                    // Remove all filters.
+                    session.getFilterChain().clear();
+                }
             }
         }