From: Jeff Trawick <trawick@apache.org>
Date: Wed, 10 Dec 2003 19:26:15 +0000 (+0000)
Subject: Fix memory leak in handling of request bodies during reverse
X-Git-Tag: pre_ajp_proxy~943
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e3c11d6cd1d46c3cec575278ae4547ea32b3b770;p=apache

Fix memory leak in handling of request bodies during reverse
proxy operations.

PR:      24991
Submitted by: Larry Toppi <larry.toppi citrix.com>
Reviewed by:  Jeff Trawick


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

diff --git a/CHANGES b/CHANGES
index 9dfd34e8c4..164d7183ff 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,9 @@ Changes with Apache 2.1.0-dev
 
   [Remove entries to the current 2.0 section below, when backported]
 
+  *) Fix memory leak in handling of request bodies during reverse
+     proxy operations.  PR 24991. [Larry Toppi <larry.toppi citrix.com>]
+
   *) Clean up httpd -V output: Instead of displaying the MPM source
      directory, display the MPM name and some MPM properties.
      [Geoffrey Young <geoff apache.org>]
diff --git a/modules/proxy/proxy_http.c b/modules/proxy/proxy_http.c
index 355094055d..aab0661390 100644
--- a/modules/proxy/proxy_http.c
+++ b/modules/proxy/proxy_http.c
@@ -687,7 +687,8 @@ apr_status_t ap_proxy_http_request(apr_pool_t *p, request_rec *r,
             }
 
             /* We can't pass this EOS to the output_filters. */
-            APR_BUCKET_REMOVE(APR_BRIGADE_LAST(input_brigade));
+            e = APR_BRIGADE_LAST(input_brigade);
+            apr_bucket_delete(e);
         }
 
         if (send_chunks) {