Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47237

There is no need to associate the Http11NioProcessor with a socket in between requests unless there is a comet.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@783155 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/java/org/apache/coyote/http11/Http11NioProtocol.java b/java/org/apache/coyote/http11/Http11NioProtocol.java
index 8afb4c6..13f9b75 100644 (file)
--- a/java/org/apache/coyote/http11/Http11NioProtocol.java
+++ b/java/org/apache/coyote/http11/Http11NioProtocol.java
@@ -760,6 +760,8 @@ public class Http11NioProtocol implements ProtocolHandler, MBeanRegistration
                         NioEndpoint.KeyAttachment att = (NioEndpoint.KeyAttachment)socket.getAttachment(false);
                         socket.getPoller().add(socket,att.getCometOps());
                     } else {
+                        //we should not hold on to the processor objects
+                        release(socket);
                         socket.getPoller().add(socket);
                     }
                 } else {