github_mirrors/ModSecurity
3
0
Fork 1
mirror of https://github.com/owasp-modsecurity/ModSecurity.git synced 2025-09-29 19:24:29 +03:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2239 from microsoft/fix_malformed_xml_memory_leak

Browse Source 
Properly cleanup XML parser contexts upon completion
This commit is contained in:
martinhsv
2022-06-08 17:45:53 -04:00
committed by GitHub
parent dd2d3f74b6 6a5ec1ff7b
commit fc8e5586e7
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
apache2/msc_xml.c
View File

@@ -137,6 +137,13 @@ int xml_complete(modsec_rec *msr, char **error_msg) {
* Frees the resources used for XML parsing. * Frees the resources used for XML parsing.
*/ */
apr_status_t xml_cleanup(modsec_rec *msr) { apr_status_t xml_cleanup(modsec_rec *msr) {
if (msr->xml->parsing_ctx != NULL) {
if (msr->xml->parsing_ctx->myDoc) {
xmlFreeDoc(msr->xml->parsing_ctx->myDoc);
}
xmlFreeParserCtxt(msr->xml->parsing_ctx);
msr->xml->parsing_ctx = NULL;
}
if (msr->xml->doc != NULL) { if (msr->xml->doc != NULL) {
xmlFreeDoc(msr->xml->doc); xmlFreeDoc(msr->xml->doc);
msr->xml->doc = NULL; msr->xml->doc = NULL;