Bug #14522 Updated: Crash with using $node->unlink();

ID: 14522 Updated by: jtate@php.net Reported By: klebermass@limtec.de Status: Closed Bug Type: DOM XML related Operating System: SuSe 7.3 PHP Version: CVS Download New Comment: I've commented out the offending lines in the source. You can pull it from CVS at any time. I'll also see if we can get it into the 4.2.0 release. Thanks for bringing it up again. Previous Comments: ------------------------------------------------------------------------ [2002-03-07 17:59:06] lukas@edeal.de beware: the bug is marked as closed, but a fix has not been applied to cvs yet. ------------------------------------------------------------------------ [2002-02-19 15:16:27] klebermass@limtec.de The patch from lukas@edeal.de worked on my Server. Hope it will be soon in the CVS Version ------------------------------------------------------------------------ [2002-02-19 13:43:50] lukas@edeal.de This patch against latest CVS should fix it: (basically it makes domxml not free a node on unlink and rather wait for request shutdown to free a node). btw, this patch also enables a domxml user to re-add a previously unlink()ed node, which is impossible with the current code. regards, -lukas Index: ext/domxml/php_domxml.c =================================================================== RCS file: /repository/php4/ext/domxml/php_domxml.c,v retrieving revision 1.116 diff -u -r1.116 php_domxml.c --- ext/domxml/php_domxml.c 4 Feb 2002 21:07:39 -0000 1.116 +++ ext/domxml/php_domxml.c 19 Feb 2002 18:38:57 -0000 @@ -1801,8 +1801,6 @@ DOMXML_PARAM_NONE(nodep, id, le_domxmlnodep); xmlUnlinkNode(nodep); - xmlFreeNode(nodep); - zval_dtor(id); /* This is not enough because the children won't be deleted */ } /* }}} */ ------------------------------------------------------------------------ [2002-02-19 12:23:38] klebermass@limtec.de Nothing happening about the bug? Un Downloading regularly the newest CVS Version but doesnt help ------------------------------------------------------------------------ [2002-02-12 16:22:20] rrichards@digarc.com We have also reproduced this error on both RedHat 7.2 and FreeBSD 4.4 both running PHP 4.1.1. It completely takes down FREEBSD as apache continues to log seg faults until the drive is full. We had the same results with 4.0.6 prior to the upgrade as well. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/14522 -- Edit this bug report at http://bugs.php.net/?id=14522&edit=1