DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=39605>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=39605





------- Additional Comments From [EMAIL PROTECTED]  2006-05-18 20:26 -------
If c->aborted gets set unconditionally as your patch proposes and things still
do not work I think this is a bug of the caller (seems to be subversion in your
case) as it should not call ap_pass_brigade again if c->aborted is set. But
maybe the following patch is a useful sanity check to ensure that no further
data gets sent:

Index: core_filters.c
===================================================================
--- core_filters.c      (Revision 406722)
+++ core_filters.c      (Arbeitskopie)
@@ -542,6 +542,20 @@
     apr_read_type_e eblock = APR_NONBLOCK_READ;
     apr_pool_t *input_pool = b->p;

+    /*
+     * Maybe we have been called by a buggy part in the lower filter chain
+     * or a buggy handler on a connection that has been already aborted.
+     * Just destroy the brigade and return with APR_SUCCESS.
+     */
+    if (c->aborted) {
+        ap_log_cerror(APLOG_MARK, APLOG_ERR, rv, c, "core_output_filter:"
+                      " Trying to write on a connection aborted by the"
+                      " client.");
+        if (b)
+            apr_brigade_destroy(b);
+        return APR_SUCCESS;
+    }
+
     if (ctx == NULL) {
         ctx = apr_pcalloc(c->pool, sizeof(*ctx));
         net->out_ctx = ctx;




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to