You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by ja...@apache.org on 2018/05/27 20:48:00 UTC
svn commit: r1832351 - /httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c
Author: jailletc36
Date: Sun May 27 20:48:00 2018
New Revision: 1832351
URL: http://svn.apache.org/viewvc?rev=1832351&view=rev
Log:
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
Modified:
httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c
Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c?rev=1832351&r1=1832350&r2=1832351&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c (original)
+++ httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c Sun May 27 20:48:00 2018
@@ -324,6 +324,7 @@ static int ap_proxy_ajp_request(apr_pool
* Close it to clean things up.
*/
conn->close = 1;
+ apr_brigade_destroy(input_brigade);
return HTTP_BAD_REQUEST;
}
}