Author: fhanik
Date: Tue Jun  9 22:14:55 2009
New Revision: 783155

URL: http://svn.apache.org/viewvc?rev=783155&view=rev
Log:
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.


Modified:
    tomcat/trunk/java/org/apache/coyote/http11/Http11NioProtocol.java

Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11NioProtocol.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/Http11NioProtocol.java?rev=783155&r1=783154&r2=783155&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/Http11NioProtocol.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http11/Http11NioProtocol.java Tue Jun  
9 22:14:55 2009
@@ -760,6 +760,8 @@
                         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 {



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

Reply via email to