Use ap_pass_brigade_fchk()

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

diff --git a/modules/cache/mod_cache.c b/modules/cache/mod_cache.c
index 9b655dc9d0875737c0e06a1cdbd0c4f2639bbe9e..6b4e13e8a64c5d69ae4bfb2f54b582ff2b3bd99f 100644 (file)
--- a/modules/cache/mod_cache.c
+++ b/modules/cache/mod_cache.c
@@ -292,19 +292,11 @@ static int cache_quick_handler(request_rec *r, int lookup)
     out = apr_brigade_create(r->pool, r->connection->bucket_alloc);
     e = apr_bucket_eos_create(out->bucket_alloc);
     APR_BRIGADE_INSERT_TAIL(out, e);
-    rv = ap_pass_brigade(r->output_filters, out);
-    if (rv != APR_SUCCESS) {
-        if (rv != AP_FILTER_ERROR) {
-            /* no way to know what type of error occurred */
-            ap_log_rerror(APLOG_MARK, APLOG_DEBUG, rv, r,
-                          "cache_quick_handler(%s): ap_pass_brigade returned %i",
-                          cache->provider_name, rv);
-            return HTTP_INTERNAL_SERVER_ERROR;
-        }
-        return rv;
-    }
 
-    return OK;
+    return ap_pass_brigade_fchk(r, out,
+                                apr_psprintf(r->pool,
+                                             "cache_quick_handler(%s): ap_pass_brigade returned",
+                                             cache->provider_name));
 }
 
 /**
@@ -576,19 +568,10 @@ static int cache_handler(request_rec *r)
     out = apr_brigade_create(r->pool, r->connection->bucket_alloc);
     e = apr_bucket_eos_create(out->bucket_alloc);
     APR_BRIGADE_INSERT_TAIL(out, e);
-    rv = ap_pass_brigade(r->output_filters, out);
-    if (rv != APR_SUCCESS) {
-        if (rv != AP_FILTER_ERROR) {
-            /* no way to know what type of error occurred */
-            ap_log_rerror(APLOG_MARK, APLOG_DEBUG, rv, r,
-                          "cache_handler(%s): ap_pass_brigade returned %i",
-                          cache->provider_name, rv);
-            return HTTP_INTERNAL_SERVER_ERROR;
-        }
-        return rv;
-    }
-
-    return OK;
+    return ap_pass_brigade_fchk(r, out,
+                                apr_psprintf(r->pool,
+                                             "cache(%s): ap_pass_brigade returned",
+                                             cache->provider_name));
 }
 
 /*

diff --git a/modules/mappers/mod_negotiation.c b/modules/mappers/mod_negotiation.c
index f8668db593ef355269a0b7056c308c982907f712..20fef01c566017c1d3e5def6ccde4cfdbbc8f68f 100644 (file)
--- a/modules/mappers/mod_negotiation.c
+++ b/modules/mappers/mod_negotiation.c
@@ -3051,16 +3051,7 @@ static int handle_map_file(request_rec *r)
         e = apr_bucket_eos_create(c->bucket_alloc);
         APR_BRIGADE_INSERT_TAIL(bb, e);
 
-        rv = ap_pass_brigade(r->output_filters, bb);
-        if (rv != APR_SUCCESS) { 
-            if (rv != AP_FILTER_ERROR) {
-                ap_log_rerror(APLOG_MARK, APLOG_DEBUG, rv, r,
-                              "ap_pass_brigade returned %d", rv);
-                return HTTP_INTERNAL_SERVER_ERROR;
-            }
-            return AP_FILTER_ERROR;
-        }
-        return OK;
+        return ap_pass_brigade_fchk(r, bb, NULL);
     }
 
     if (r->path_info && *r->path_info) {