2011/10/20  <ma...@apache.org>:
> Author: markt
> Date: Wed Oct 19 20:55:39 2011
> New Revision: 1186485
>
> URL: http://svn.apache.org/viewvc?rev=1186485&view=rev
> Log:
> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=52055
> Buffers not correctly reset between keep-alive requests when using async
>
> Modified:
>    tomcat/tc7.0.x/trunk/   (props changed)
>    
> tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java
>    tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
>

> --- 
> tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java
>  (original)
> +++ 
> tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java
>  Wed Oct 19 20:55:39 2011
> @@ -1538,6 +1538,8 @@ public abstract class AbstractHttp11Proc
>             if (!keepAlive) {
>                 return SocketState.CLOSED;
>             } else {
> +                getInputBuffer().nextRequest();
> +                getOutputBuffer().nextRequest();
>                 return SocketState.OPEN;
>             }
>         }
>


Searching for places that use org.apache.coyote.Constants.STAGE_ENDED ...

1) it looks like AbstractAjpProcessor#asyncDispatch(SocketStatus)
needs similar patch:
it has "return SocketState.OPEN;"

If I understand correctly in Ajp that will be recycle(false)

2) In Http11NioProcessor#event() I do not see nextRequest() calls, but
similar implementation in Http11AprProcessor#event() has nextRequest()
calls.


Best regards,
Konstantin Kolinko

> --- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
> +++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Wed Oct 19 20:55:39 2011
> @@ -110,6 +110,11 @@
>         authentication from being lost during the authentication process.
>         (markt)
>       </fix>
> +      <fix>
> +        <bug>52055</bug>: Ensure that the input and output buffers are 
> correctly
> +        reset between keep-alive requests when using Servlet 3.0 asynchronous
> +        request processing. (markt)
> +      </fix>
>     </changelog>
>   </subsection>
>   <subsection name="Coyote">
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>
>

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

Reply via email to