Re: cvs: php4 /main streams.c

I thought there was another error somewhere but it seems it was the efree and then requesting the memory. Without the patch i simply got "*RECURSION*" out of php. Anybody able to explain this? marcus At 02:24 23.11.2002, Marcus Boerger wrote:

helly           Fri Nov 22 20:24:08 2002 EDT

Modified files:

    /php4/main  streams.c

Log: MFB: Fix memleak in debug mode Index: php4/main/streams.c diff -u php4/main/streams.c:1.130 php4/main/streams.c:1.131 --- php4/main/streams.c:1.130 Mon Nov 18 12:23:21 2002 +++ php4/main/streams.c Fri Nov 22 20:24:08 2002 @@ -20,7 +20,7 @@

    +----------------------------------------------------------------------+

*/ -/* $Id: streams.c,v 1.130 2002/11/18 17:23:21 helly Exp $ */ +/* $Id: streams.c,v 1.131 2002/11/23 01:24:08 helly Exp $ */ #define _GNU_SOURCE #include "php.h" @@ -2317,7 +2317,7 @@

                                return stream;
                        case PHP_STREAM_RELEASED:

#if ZEND_DEBUG

-                               newstream->__orig_path = copy_of_path;
+                               newstream->__orig_path = estrdup(copy_of_path);

#endif

                                return newstream;
                        default:

-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php