Re: cvs: php4 /ext/dom element.c node.c php_dom.c
| From: | Moriyoshi Koizumi | Date: | Thu, 05 Jun 2003 19:20:47 +0000 |
| Subject: | Re: cvs: php4 /ext/dom element.c node.c php_dom.c | ||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-21558@lists.php.net to get a copy of this message | ||
Sterling Hughes <sterling@bumblebury.com> wrote:
> 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. :)
:) What's the script like that Rob sent to you? Hopefully I can help..
Moriyoshi
> -sterling
>
> >
> > Moriyoshi
> --
> "People can have the Model T in any colour -- so long as it's black."
> - Henry Ford
>
> --
> PHP CVS Mailing List (http://www.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>