From: Joe Watkins <krakjoe@php.net>
Date: Sat, 14 May 2016 07:10:16 +0000 (+0100)
Subject: fix #72206 (xml_parser_create/xml_parser_free leaks mem)
X-Git-Tag: php-7.0.8RC1~60
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b165114013b690b338a83c606974540e0830c2bd;p=php

fix #72206 (xml_parser_create/xml_parser_free leaks mem)
---

diff --git a/NEWS b/NEWS
index e9cc944bf3..d3cd2f5daf 100644
--- a/NEWS
+++ b/NEWS
@@ -17,6 +17,8 @@ PHP                                                                        NEWS
   . Fixed bug #72193 (dns_get_record returns array containing elements of
     type 'unknown'). (Laruence)
 
+- XML:
+  . Fixed #72206 (xml_parser_create/xml_parser_free leaks mem). (Joe)
 
 26 May 2016 PHP 7.0.7
 
diff --git a/ext/xml/xml.c b/ext/xml/xml.c
index 6c5c9e39b2..d788b9b0b8 100644
--- a/ext/xml/xml.c
+++ b/ext/xml/xml.c
@@ -1573,9 +1573,10 @@ PHP_FUNCTION(xml_parser_free)
 		RETURN_FALSE;
 	}
 
-	res = Z_RES(parser->index);
-	ZVAL_UNDEF(&parser->index);
-	zend_list_close(res);
+	if (zend_list_delete(Z_RES(parser->index)) == FAILURE) {
+		RETURN_FALSE;
+	}
+
 	RETURN_TRUE;
 }
 /* }}} */