You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by rj...@apache.org on 2010/06/09 14:44:01 UTC

svn commit: r952980 - in /tomcat/jk/trunk: native/common/jk_ajp_common.c xdocs/miscellaneous/changelog.xml

Author: rjung
Date: Wed Jun  9 12:44:01 2010
New Revision: 952980

URL: http://svn.apache.org/viewvc?rev=952980&view=rev
Log:
Fx BZ49413: Drop AJP13 flush packets send by the backend
after the response has been finished.

Modified:
    tomcat/jk/trunk/native/common/jk_ajp_common.c
    tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml

Modified: tomcat/jk/trunk/native/common/jk_ajp_common.c
URL: http://svn.apache.org/viewvc/tomcat/jk/trunk/native/common/jk_ajp_common.c?rev=952980&r1=952979&r2=952980&view=diff
==============================================================================
--- tomcat/jk/trunk/native/common/jk_ajp_common.c (original)
+++ tomcat/jk/trunk/native/common/jk_ajp_common.c Wed Jun  9 12:44:01 2010
@@ -1835,6 +1835,16 @@ static int ajp_process_callback(jk_msg_b
 
     case JK_AJP13_SEND_BODY_CHUNK:
         if (ae->last_op == JK_AJP13_FORWARD_REQUEST) {
+            /* AJP13_SEND_BODY_CHUNK with length 0 is
+             * explicit flush packet message.
+             * Ignore those if they are left over from previous responses.
+             * Reportedly some versions of JBoss suffer from that problem.
+             */
+            if (jk_b_get_int(msg) == 0) {
+                jk_log(l, JK_LOG_DEBUG,
+                       "Ignoring flush message received while sending the request");
+                return ae->last_op;
+            }
             /* We have just send a request but received something
              * that probably originates from buffered response.
              */

Modified: tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml?rev=952980&r1=952979&r2=952980&view=diff
==============================================================================
--- tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml (original)
+++ tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml Wed Jun  9 12:44:01 2010
@@ -44,6 +44,10 @@
   <br />
   <subsection name="Native">
     <changelog>
+      <fix>
+        <bug>49413</bug>: AJP13: Drop flush packets send by the backend
+        after the response has been finished. (rjung)
+      </fix>
       <update>
         Apache: Added version number resource for mod_jk.so on Windows. (timw)
       </update>



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