tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From fha...@apache.org
Subject svn commit: r783155 - /tomcat/trunk/java/org/apache/coyote/http11/Http11NioProtocol.java
Date Tue, 09 Jun 2009 22:14:55 GMT
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-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message