NULL pointers can be passed to free_fcn. This is the correct behaviour.

diff --git a/ext/xml/xml.c b/ext/xml/xml.c
index 83f377140a593cc49c5371c2ee059e944008b551..8b3306dd573bc65a40ce1755ffe801c938aecd84 100644 (file)
--- a/ext/xml/xml.c
+++ b/ext/xml/xml.c
@@ -192,7 +192,9 @@ static void *php_xml_realloc_wrapper(void *ptr, size_t sz)
 
 static void php_xml_free_wrapper(void *ptr)
 {
-       efree(ptr);
+       if (ptr != NULL) {
+               efree(ptr);
+       }
 }
 
 PHP_MINIT_FUNCTION(xml)