Please provide a short, self-containing, reproduceable, easy copy&paste script.

Feedback.

Please try it with a snapshot from snaps.php.net too...

Derick

No feedback. Closing.

Sorry i havent been at home the second part of December.

Here a sample code:

<?php
$xmlstruc=xmldoc("<?xml version=\"1.0\" standalone=\"no\" ?><root><down>bla</down></root>");

$xmlnode=$xmlstruc->root();
        $xmlchilds=$xmlnode->children();
        if(!is_array($xmlchilds)) $xmlchilds=array($xmlchilds);
        foreach ( $xmlchilds as $workingchild) {
                        $workingchild->unlink();}

echo $xmlstruc->dumpmem();


?>


martin

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.

Nothing happening about the bug? Un Downloading regularly the newest CVS Version but doesnt help

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 */
 }
 /* }}} */

The patch from lukas@edeal.de worked on my Server.

Hope it will be soon in the CVS Version 

beware: the bug is marked as closed, but a fix has not been applied to cvs yet.

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.

The script posted here does not crash for me with the latest CVS (27-03-2002). Can you try a snapshot from snaps.php.net?

Derick

derick,

as of latest CVS (HEAD, php_domxml.c rev.1.120) a fix similar to the patch within this bug's history has already been committed by jtate (rev.1.119 was the last to crash on me)

if you tested with latest head cvs, it's no wonder you dont see the php barf ;)

having the same problem with php 4.1.2 on w2k. where's the fix?

It's fixed in CVS, so closing. This fix will be in version 4.2.0

Derick