]> granicus.if.org Git - apache/commitdiff
core: follow up to r1836237: core filter's tmp_flush_bb not used anymore.
authorYann Ylavic <ylavic@apache.org>
Fri, 20 Jul 2018 15:04:55 +0000 (15:04 +0000)
committerYann Ylavic <ylavic@apache.org>
Fri, 20 Jul 2018 15:04:55 +0000 (15:04 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1836354 13f79535-47bb-0310-9956-ffa450edef68

server/core_filters.c

index 5e5d5cf149ca808937df5492f96f01e4532eebb0..02256e90b3159eef580830efc923ced6f841007f 100644 (file)
@@ -78,7 +78,6 @@ do { \
 #define APLOG_MODULE_INDEX AP_CORE_MODULE_INDEX
 
 struct core_output_filter_ctx {
-    apr_bucket_brigade *tmp_flush_bb;
     apr_bucket_brigade *empty_bb;
     apr_size_t bytes_written;
     struct iovec *vec;
@@ -370,12 +369,6 @@ apr_status_t ap_core_output_filter(ap_filter_t *f, apr_bucket_brigade *bb)
     if (ctx == NULL) {
         ctx = apr_pcalloc(c->pool, sizeof(*ctx));
         net->out_ctx = (core_output_filter_ctx_t *)ctx;
-        /*
-         * Need to create tmp brigade with correct lifetime. Passing
-         * NULL to apr_brigade_split_ex would result in a brigade
-         * allocated from bb->pool which might be wrong.
-         */
-        ctx->tmp_flush_bb = apr_brigade_create(c->pool, c->bucket_alloc);
     }
 
     /* remain compatible with legacy MPMs that passed NULL to this filter */
@@ -383,6 +376,9 @@ apr_status_t ap_core_output_filter(ap_filter_t *f, apr_bucket_brigade *bb)
         if (ctx->empty_bb == NULL) {
             ctx->empty_bb = apr_brigade_create(c->pool, c->bucket_alloc);
         }
+        else {
+            apr_brigade_cleanup(ctx->empty_bb);
+        }
         bb = ctx->empty_bb;
     }