fix the case when a server doesn't send a response string

Submitted by:	Brett Hutley <brett@hutley.net> J.D. Silvester <jsilves@uwo.ca>

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

diff --git a/CHANGES b/CHANGES
index 29bc4e5c3c4919420419b347e0d7e611635b171a..b50045ac6a0d2a70a86b8465d34d68e16b69c4b1 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,4 +1,6 @@
 Changes with Apache 2.0.41
+  *) mod-proxy: handle proxied responses with no status lines
+     [JD Silvester <jsilves@uwo.ca>, Brett Huttley <brett@huttley.net>]
 
   *) Fix bug where environment or command line arguments containing 
      non-ASCII-7 characters would cause the Win32 child process creation

diff --git a/modules/proxy/proxy_http.c b/modules/proxy/proxy_http.c
index 372565ab749abd0959146d6fd00e9be4ff21b0cf..fe402350054a5794845a6324cb08e4b9eb847e0b 100644 (file)
--- a/modules/proxy/proxy_http.c
+++ b/modules/proxy/proxy_http.c
@@ -676,6 +676,7 @@ apr_status_t ap_proxy_http_process_response(apr_pool_t * p, request_rec *r,
                                             char *server_portstr) {
     conn_rec *c = r->connection;
     char buffer[HUGE_STRING_LEN];
+    char keepchar;
     request_rec *rp;
     apr_bucket *e;
     int len, backasswards;
@@ -735,10 +736,11 @@ apr_status_t ap_proxy_http_process_response(apr_pool_t * p, request_rec *r,
             }
             backasswards = 0;
 
+            keepchar = buffer[12];
             buffer[12] = '\0';
             r->status = atoi(&buffer[9]);
 
-            buffer[12] = ' ';
+            buffer[12] = keepchar;
             r->status_line = apr_pstrdup(p, &buffer[9]);
 
             /* read the headers. */