Re: cvs: php4 /ext/dom element.c node.c php_dom.c
| From: | Sterling Hughes | Date: | Thu, 05 Jun 2003 19:08:51 +0000 |
| Subject: | Re: cvs: php4 /ext/dom element.c node.c php_dom.c | ||
| References: | 1 2 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-21556@lists.php.net to get a copy of this message | ||
On Thu, 2003-06-05 at 15:16, Moriyoshi Koizumi wrote:
> The following part seems to cause memleaks. Is this really a right
> fix?
>
> > + if (!PZVAL_IS_REF(node)) {
> > + zval_add_ref(&node);
> > + }
> > +
>
> /home/koizumi/src/php5/ext/dom/php_dom.c(814) : Freeing 0x0831D19C (32 bytes),
> script=/home/koizumi/src/php5/ext/dom/tests/dom001.php
> /home/koizumi/src/php5/Zend/zend_hash.c(150) : Actual location (location was rel
> ayed)
> /home/koizumi/src/php5/ext/dom/php_dom.c(813) : Freeing 0x0831D13C (44 bytes),
> script=/home/koizumi/src/php5/ext/dom/tests/dom001.php
> /home/koizumi/src/php5/ext/dom/php_dom.c(798) : Freeing 0x08314B24 (24 bytes),
> script=/home/koizumi/src/php5/ext/dom/tests/dom001.php
>
Yep. But it doesn't crash. :)
-sterling
>
> Moriyoshi
--
"People can have the Model T in any colour -- so long as it's black."
- Henry Ford