From: Jim Jagielski <jim@apache.org>
Date: Thu, 17 Apr 2008 14:07:25 +0000 (+0000)
Subject: Merge r644391 from trunk:
X-Git-Tag: 2.2.9~242
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5b4a0d935ff2feb47eff1493f24fba2890a4aa6c;p=thirdparty%2Fapache%2Fhttpd.git

Merge r644391 from trunk:

close PR 44381

Reviewed by: jim


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x@649116 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/CHANGES b/CHANGES
index b14e7d71dc5..2963c47bb57 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache 2.2.9
 
+  *) http_filters: Don't spin if get an error when reading the
+     next chunk. PR 44381 [Ruediger Pluem]
+
   *) ab: Do not try to read non existing response bodies of HEAD requests.
      PR 34275 [Takashi Sato <serai lans-tv.com>]
 
diff --git a/STATUS b/STATUS
index 06080e12e6c..2d4affcf8d2 100644
--- a/STATUS
+++ b/STATUS
@@ -88,15 +88,6 @@ RELEASE SHOWSTOPPERS:
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
   [ start all new proposals below, under PATCHES PROPOSED. ]
 
-  * http_filter: Don't spin if we get an error when reading
-    (the next) chunk.
-    PR: 44381
-   Trunk version of patch: 
-       http://svn.apache.org/viewvc?view=rev&revision=644391
-   Backport version for 2.2.x of patch:
-       Trunk version of patch works
-   +1: jim, rpluem, covener
-
  * mod_substitute: Make default behavior flattening of buckets. Allow
    for people to specifically set "quick" mode.
       Trunk version of patch:
diff --git a/modules/http/http_filters.c b/modules/http/http_filters.c
index 83e2f784261..83e8fb34328 100644
--- a/modules/http/http_filters.c
+++ b/modules/http/http_filters.c
@@ -425,6 +425,10 @@ apr_status_t ap_http_filter(ap_filter_t *f, apr_bucket_brigade *b,
                           (APR_STATUS_IS_EAGAIN(rv)) )) {
                         return APR_EAGAIN;
                     }
+                    /* If we get an error, then leave */
+                    if (rv != APR_SUCCESS) {
+                        return rv;
+                    }
                     /*
                      * We really don't care whats on this line. If it is RFC
                      * compliant it should be only \r\n. If there is more