Ok. Thanks for taking a look.

Potenial problem could be the following line:

const char * objName =ZSTR_VAL(Z_OBJCE_P(oth)->name);


I think I need to free the objName some how.


> On 22 Sep 2017, at 18:42, Sara Golemon <poll...@php.net> wrote:
> 
>>> On Fri, Sep 22, 2017 at 12:21 PM, O'Neil Delpratt <on...@saxonica.com> 
>>> wrote:
>>> I still have the memory leak without that line. I think I was thinking I 
>>> need to add a refcount to the XdmNode object, but maybe I am wrong.
>>> 
>> The Following gives leak:
>> 
>> $proc = Saxon/SaxonProcessor
>> 
>> $xquery = $proc->NewXQueryProcessor()
>> 
>> $sourceNode = $proc->parseXmlFromString("<foo />”);
>> 
>> $xquery->setContextItem($sourceNode); // with this line commented out there 
>> is no leak.
>> 
> Okay, well the bad news is that assuming the extension you're working
> with is the link you gave initially and you did remove the
> Z_ADDREF_P() from XQueryProcessor::setContextItem()... I can't point
> to why you'd have a memory leak.
> 
> -Sara



--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to