From: Jeff Trawick <trawick@apache.org>
Date: Tue, 20 Mar 2001 21:40:50 +0000 (+0000)
Subject: Empty out the brigade shared by ap_getline()/ap_get_client_block()
X-Git-Tag: 2.0.15~18
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5f6998ec208abb168324a1960b30a07ee39b676c;p=apache

Empty out the brigade shared by ap_getline()/ap_get_client_block()
on error exit from ap_getline().  Some other code got upset because
the wrong data was in the brigade.


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

diff --git a/CHANGES b/CHANGES
index 81d4a1dcd8..2618ad7efc 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,5 +1,9 @@
 Changes with Apache 2.0.15-dev
 
+  *) Empty out the brigade shared by ap_getline()/ap_get_client_block()
+     on error exit from ap_getline().  Some other code got upset because
+     the wrong data was in the brigade.  [Greg Ames, Jeff Trawick]
+
   *) Handle ap_discard_request_body() being called more than once.
      [Greg Ames, Jeff Trawick]
 
diff --git a/server/protocol.c b/server/protocol.c
index a58389a05a..86a01e91e4 100644
--- a/server/protocol.c
+++ b/server/protocol.c
@@ -519,8 +519,12 @@ AP_CORE_DECLARE(int) ap_getline(char *s, int n, request_rec *r, int fold)
 
     while (1) {
         if (APR_BRIGADE_EMPTY(b)) {
-            if (ap_get_brigade(c->input_filters, b, AP_MODE_BLOCKING) != APR_SUCCESS ||
+            if ((retval = ap_get_brigade(c->input_filters, b, AP_MODE_BLOCKING)) != APR_SUCCESS ||
                 APR_BRIGADE_EMPTY(b)) {
+                apr_brigade_destroy(b);
+                if (retval != APR_EOF && retval != APR_TIMEUP) {
+                    ap_log_rerror(APLOG_MARK, APLOG_ERR, retval, r, "ap_get_brigade() failed");
+                }
                 return -1;
             }
         }
@@ -530,10 +534,15 @@ AP_CORE_DECLARE(int) ap_getline(char *s, int n, request_rec *r, int fold)
             continue;
         }
         retval = apr_bucket_read(e, &temp, &length, APR_BLOCK_READ);
-
         if (retval != APR_SUCCESS) {
-            total = ((length < 0) && (total == 0)) ? -1 : total;
-            break;
+            apr_brigade_destroy(b);
+            ap_log_rerror(APLOG_MARK, APLOG_ERR, retval, r, "apr_bucket_read() failed");
+            if (total) {
+                break; /* report previously-read data to caller, do ap_xlate_proto_to_ascii() */
+            }
+            else {
+                return -1;
+            }
         }
 
         if ((looking_ahead) && (*temp != APR_ASCII_BLANK) && (*temp != APR_ASCII_TAB)) { 
@@ -552,12 +561,12 @@ AP_CORE_DECLARE(int) ap_getline(char *s, int n, request_rec *r, int fold)
         else {
             /* input line was larger than the caller's buffer */
             apr_brigade_destroy(b); 
-            
+
             /* don't need to worry about req_cfg->bb being bogus.
              * the request is about to die, and ErrorDocument
              * redirects get a new req_cfg->bb
              */
-            
+
             return -1;
         }