# Fw: cvs: php-src(PHP_5_2) /main/streams streams.c

- Thread: https://externals.io/message/24175
- Original archive: https://news-web.php.net/php.internals/24175
- Started by Nuno Lopes on 2006-06-21T22:30:53+00:00
- Messages: 1
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #24175 — Nuno Lopes — 2006-06-21T22:30:53+00:00

URL: https://externals.io/message/24175#24175

Not sure why this didn't get through..

----- Original Message ----- 
> Hi,
>
> > […]
>
>
> I don't think this "fix" is correct. With your patch, PHP will do more 2 
> syscalls without any reason (because PHP will try to read the file again).
> Anyway, I don't get any memory leak without your patch (at least with PHP 
> 5.2). And take a look at _php_stream_passthru() or 
> _php_stream_copy_to_stream(). If your patch is correct, these function 
> would also need to be patched. Again, I get zero leaks.
>
> Nuno