From: Christophe Jaillet <jailletc36@apache.org>
Date: Sun, 27 May 2018 20:48:00 +0000 (+0000)
Subject: All error handling paths of this function call 'apr_brigade_destroy()' , except this... 
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=07ffca3af4d2a8ae6fa512c35af05119301c418a;p=apache

All error handling paths of this function call 'apr_brigade_destroy()' , except this one.
So add it here too.

Probably spotted with the help of the Coccinelle software (Thx Julia for the patch and for Coccinelle)

See PR 53016

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

diff --git a/modules/proxy/mod_proxy_ajp.c b/modules/proxy/mod_proxy_ajp.c
index 843f3995ed..3ffa1b554d 100644
--- a/modules/proxy/mod_proxy_ajp.c
+++ b/modules/proxy/mod_proxy_ajp.c
@@ -324,6 +324,7 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r,
              * Close it to clean things up.
              */
             conn->close = 1;
+            apr_brigade_destroy(input_brigade);
             return HTTP_BAD_REQUEST;
         }
     }