Fix for bug #24219 segfault during resource destruction

Used bug fix to optimize free_doc routine

diff --git a/ext/domxml/php_domxml.c b/ext/domxml/php_domxml.c
index a20abdc9330e5c51f0d1ba11375818e1941803f9..a00ecb947e5c0c0c4eea67062a0dad2ab6212ea7 100644 (file)
--- a/ext/domxml/php_domxml.c
+++ b/ext/domxml/php_domxml.c
@@ -729,7 +729,7 @@ static void php_free_xml_doc(zend_rsrc_list_entry *rsrc TSRMLS_DC)
        xmlDoc *doc = (xmlDoc *) rsrc->ptr;
 
        if (doc) {
-               node_list_wrapper_dtor(doc->children, 0 TSRMLS_CC);
+               node_list_wrapper_dtor(doc->children, 1 TSRMLS_CC);
                node_wrapper_dtor((xmlNodePtr) doc);
                xmlFreeDoc(doc);
        }
@@ -745,8 +745,8 @@ static void php_free_xml_node(zend_rsrc_list_entry *rsrc TSRMLS_DC)
                /* Attribute Nodes ccontain accessible children 
                attr_list_wrapper_dtor(node->properties); */
                xmlSetTreeDoc(node, NULL);
-               node_list_wrapper_dtor((xmlNodePtr) node->properties, 0 TSRMLS_CC);
-               node_list_wrapper_dtor(node->children, 0 TSRMLS_CC);
+               node_list_wrapper_dtor((xmlNodePtr) node->properties, 1 TSRMLS_CC);
+               node_list_wrapper_dtor(node->children, 1 TSRMLS_CC);
                node_wrapper_dtor(node);
                xmlFreeNode(node);
        } else {
@@ -759,7 +759,7 @@ static void php_free_xml_attr(zend_rsrc_list_entry *rsrc TSRMLS_DC)
        xmlNodePtr node = (xmlNodePtr) rsrc->ptr;
        if (node->parent == NULL) {
                /* Attribute Nodes contain accessible children */
-               node_list_wrapper_dtor(node->children, 0 TSRMLS_CC);
+               node_list_wrapper_dtor(node->children, 1 TSRMLS_CC);
                node_wrapper_dtor(node);
                xmlFreeProp((xmlAttrPtr) node);
        } else {
@@ -4488,10 +4488,10 @@ PHP_FUNCTION(domxml_doc_free_doc)
                RETURN_FALSE;
        }
 
+       /* No need to do this as php_free_xml_doc will kill the children
        node_list_wrapper_dtor(docp->children, 1 TSRMLS_CC);
        node_list_wrapper_dtor((xmlNodePtr) docp->properties, 1 TSRMLS_CC);
-       /* Attribute Nodes ccontain accessible children 
-       attr_list_wrapper_dtor(docp->properties); */
+       */
        node_wrapper_free(docp TSRMLS_CC);
 
        RETURN_TRUE;