From: Stefan Fritsch <sf@apache.org>
Date: Mon, 9 Apr 2012 09:28:14 +0000 (+0000)
Subject: If there is an error reading the headers from the backend,
X-Git-Tag: 2.5.0-alpha~7231
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=555c89745f728d7c1894ed0ce359d72688ea26e6;p=apache

If there is an error reading the headers from the backend,
send an error to the client


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

diff --git a/CHANGES b/CHANGES
index 9172a1668a..dbf53664ed 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache 2.5.0
 
+  *) mod_proxy_fcgi: If there is an error reading the headers from the
+     backend, send an error to the client. [Stefan Fritsch]
+
   *) mod_rewrite: Fix RewriteCond integer checks to be parsed correctly.
      PR 53023. [Axel Reinhold <apache freakout.de>, André Malo]
 
diff --git a/modules/proxy/mod_proxy_fcgi.c b/modules/proxy/mod_proxy_fcgi.c
index d0df5fa12f..0f844163dc 100644
--- a/modules/proxy/mod_proxy_fcgi.c
+++ b/modules/proxy/mod_proxy_fcgi.c
@@ -748,10 +748,10 @@ recv_again:
                                 apr_brigade_cleanup(ob);
                                 tmp_b = apr_bucket_eos_create(c->bucket_alloc);
                                 APR_BRIGADE_INSERT_TAIL(ob, tmp_b);
+                                r->status = status;
                                 ap_pass_brigade(r->output_filters, ob);
                                 ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, APLOGNO(01070)
                                               "Error parsing script headers");
-                                r->status = status;
                                 rv = APR_EINVAL;
                                 break;
                             }