Fixed memory leak

dstogov · dstogov · commit 61e9b0d26536 · 2014-04-15T15:50:34.000+04:00
diff --git a/ext/libxml/libxml.c b/ext/libxml/libxml.c
@@ -728,6 +728,10 @@ PHP_LIBXML_API void php_libxml_error_handler(void *ctx, const char *msg, ...)
 	va_end(args);
 }
 
+static void php_libxml_exports_dtor(zval *zv)
+{
+	free(Z_PTR_P(zv));
+}
 
 PHP_LIBXML_API void php_libxml_initialize(void)
 {
@@ -738,7 +742,7 @@ PHP_LIBXML_API void php_libxml_initialize(void)
 		_php_libxml_default_entity_loader = xmlGetExternalEntityLoader();
 		xmlSetExternalEntityLoader(_php_libxml_pre_ext_ent_loader);
 
-		zend_hash_init(&php_libxml_exports, 0, NULL, NULL, 1);
+		zend_hash_init(&php_libxml_exports, 0, NULL, php_libxml_exports_dtor, 1);
 
 		_php_libxml_initialized = 1;
 	}

Original file line number	Diff line number	Diff line change
`@@ -728,6 +728,10 @@ PHP_LIBXML_API void php_libxml_error_handler(void ctx, const char msg, ...)`
`728`	`728`	`va_end(args);`
`729`	`729`	`}`
`730`	`730`
	`731`	`+static void php_libxml_exports_dtor(zval *zv)`
	`732`	`+{`
	`733`	`+ free(Z_PTR_P(zv));`
	`734`	`+}`
`731`	`735`
`732`	`736`	`PHP_LIBXML_API void php_libxml_initialize(void)`
`733`	`737`	`{`
`@@ -738,7 +742,7 @@ PHP_LIBXML_API void php_libxml_initialize(void)`
`738`	`742`	`_php_libxml_default_entity_loader = xmlGetExternalEntityLoader();`
`739`	`743`	`xmlSetExternalEntityLoader(_php_libxml_pre_ext_ent_loader);`
`740`	`744`
`741`		`- zend_hash_init(&php_libxml_exports, 0, NULL, NULL, 1);`
	`745`	`+ zend_hash_init(&php_libxml_exports, 0, NULL, php_libxml_exports_dtor, 1);`
`742`	`746`
`743`	`747`	`_php_libxml_initialized = 1;`
`744`	`748`	`}`