https://bz.apache.org/bugzilla/show_bug.cgi?id=64234

--- Comment #1 from Ruediger Pluem <[email protected]> ---
Thanks for the patch an the analysis. Two comments:

1. We need to have a patch against trunk. This will later be backported to
2.4.x. I already saw that both code bases already differ in that area.
2. I cannot think of anything useful that can be contained in bb in case of an
error before we add the the error bucket and the eoc bucket. So I think the
better fix would be the following (against trunk):

Index: mod_proxy_http.c
===================================================================
--- mod_proxy_http.c    (revision 1875177)
+++ mod_proxy_http.c    (working copy)
@@ -1787,6 +1787,7 @@
                          * through a response, our only option is to
                          * disconnect the client too.
                          */
+                        apr_brigade_cleanup(bb);
                         e = ap_bucket_error_create(HTTP_GATEWAY_TIME_OUT,
NULL,
                                 r->pool, c->bucket_alloc);
                         APR_BRIGADE_INSERT_TAIL(bb, e);

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to