Author: markt
Date: Thu Aug 22 21:44:57 2013
New Revision: 1516626

URL: http://svn.apache.org/r1516626
Log:
Ensure destroy() method of upgrade handler is called

Modified:
    tomcat/tc7.0.x/trunk/java/org/apache/coyote/AbstractProtocol.java

Modified: tomcat/tc7.0.x/trunk/java/org/apache/coyote/AbstractProtocol.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/coyote/AbstractProtocol.java?rev=1516626&r1=1516625&r2=1516626&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/coyote/AbstractProtocol.java (original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/coyote/AbstractProtocol.java Thu Aug 
22 21:44:57 2013
@@ -676,8 +676,11 @@ public abstract class AbstractProtocol i
                     // Connection closed. OK to recycle the processor. Upgrade
                     // processors are not recycled.
                     connections.remove(socket);
-                    if (!(processor instanceof 
org.apache.coyote.http11.upgrade.UpgradeProcessor)
-                            && !processor.isUpgrade()) {
+                    if (processor.isUpgrade()) {
+                        processor.getHttpUpgradeHandler().destroy();
+                    } else if (processor instanceof 
org.apache.coyote.http11.upgrade.UpgradeProcessor) {
+                        // NO-OP
+                    } else {
                         release(wrapper, processor, true, false);
                     }
                 }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to