Re: cvs: php-src /ext/zip php_zip.c

Hello, On 11/12/06, Nuno Lopes <nlopess@php.net> wrote:

It sounds wrong. _zip_free is a private function and is used in zip_close. Can you do it there instead? itself.oo_delete.phpt works here, which error do you have? --Pierre Yes, I also thought that it was a private function, but I really had to use it. The problem arises when zip_close() returns -1 (that was happending with me)

oo_delete failed on your box? Weird, it works here on all my test systems. What do you use?

and that function doesn't call _zip_free() on error (so that you can retrieve the error), thus creating a memleak.

Yes I realized that now, I will add a public function to destroy a za, _zip_free must stay behind the scene :) --PIerre