]> granicus.if.org Git - apache/commitdiff
close PR 44381
authorJim Jagielski <jim@apache.org>
Thu, 3 Apr 2008 16:21:20 +0000 (16:21 +0000)
committerJim Jagielski <jim@apache.org>
Thu, 3 Apr 2008 16:21:20 +0000 (16:21 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@644391 13f79535-47bb-0310-9956-ffa450edef68

CHANGES
modules/http/http_filters.c

diff --git a/CHANGES b/CHANGES
index 2ece280a61720eedd505557b70691a537306fc18..8bf9b084cfd8a08d616f2c3574c3802478c878d2 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,9 @@
 Changes with Apache 2.3.0
 [ When backported to 2.2.x, remove entry from this file ]
 
+  *) http_filters: Don't spin if get an error when reading the
+     next chunk. PR 44381 [Ruediger Pluem]
+
   *) mod_dav: Return "method not allowed" if the destination URI of a WebDAV
      copy / move operation is no DAV resource. PR 44734 [Ruediger Pluem]
 
index 4b9bbff4b497ce8459774054ade2ca1864cd9e1d..90d30269bc687e39b8ae166def6af795affd8fb1 100644 (file)
@@ -427,6 +427,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