Re: leaks related with virtual_file_ex()
| From: | Ilia Alshanetsky | Date: | Mon, 18 Sep 2006 02:51:25 +0000 |
| Subject: | Re: leaks related with virtual_file_ex() | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-25749@lists.php.net to get a copy of this message | ||
On 17-Sep-06, at 8:03 AM, Nuno Lopes wrote:
OK, first patch to fix the 'tests/lang/bug35176.phpt' specific problem: http://mega.ist.utl.pt/~ncpl/zend_include_memleak.txt There might be other issues with virtual_file_ex() that I haven't fully tested.Can you not simply add the free before the zend_message_dispatcher(ZMSG_FAILED_REQUIRE_FOPEN, Z_STRVAL_P(inc_filename)); line which is followed by a fatal error and in all other instances rely on the existing free() ? Ilia Alshanetsky