]> granicus.if.org Git - apache/commitdiff
PR#39710 - badly broken errordocuments for CGI
authorNick Kew <niq@apache.org>
Sat, 26 May 2007 18:50:18 +0000 (18:50 +0000)
committerNick Kew <niq@apache.org>
Sat, 26 May 2007 18:50:18 +0000 (18:50 +0000)
We've just had another duplicate report of this on bugzilla.
We've got a simple patch, and people asking WTF is going on
with inaction.  Noone seems clear on why the patch shouldn't
be applied (http://marc.info/?l=apache-httpd-dev&m=117760311129386&w=2).

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@541926 13f79535-47bb-0310-9956-ffa450edef68

modules/generators/mod_cgi.c

index 2c668c6b58a81a4da0f10d8d3da061513a6db129..b0ffdabe25540ff5cd8ec64575b0979fd6f156ed 100644 (file)
@@ -929,18 +929,7 @@ static int cgi_handler(request_rec *r)
         int ret;
 
         if ((ret = ap_scan_script_header_err_brigade(r, bb, sbuf))) {
-            ret = log_script(r, conf, ret, dbuf, sbuf, bb, script_err);
-
-            /* Set our status. */
-            r->status = ret;
-
-            /* Pass EOS bucket down the filter chain. */
-            apr_brigade_cleanup(bb);
-            b = apr_bucket_eos_create(c->bucket_alloc);
-            APR_BRIGADE_INSERT_TAIL(bb, b);
-            ap_pass_brigade(r->output_filters, bb);
-
-            return ret;
+            return log_script(r, conf, ret, dbuf, sbuf, bb, script_err);
         }
 
         location = apr_table_get(r->headers_out, "Location");