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