On Mon, 23 May 2005 09:54:44 -0400
Wez Furlong <[EMAIL PROTECTED]> wrote:

> WTF!???
> 
> Please revert, immediately.

Please care to explain why.
I tested it with all kinds of wrappers and it works fine for me.

> On 5/23/05, Antony Dovgal <[EMAIL PROTECTED]> wrote:
> > tony2001                Mon May 23 07:51:53 2005 EDT
> > 
> >   Modified files:
> >     /php-src/main/streams       streams.c
> >   Log:
> >   fix bug #32810 (fread after tmpfile() reads only 8192 bytes)
> > 
> > 
> > http://cvs.php.net/diff.php/php-src/main/streams/streams.c?r1=1.78&r2=1.79&ty=u
> > Index: php-src/main/streams/streams.c
> > diff -u php-src/main/streams/streams.c:1.78 
> > php-src/main/streams/streams.c:1.79
> > --- php-src/main/streams/streams.c:1.78 Mon May 16 04:37:13 2005
> > +++ php-src/main/streams/streams.c      Mon May 23 07:51:53 2005
> > @@ -19,7 +19,7 @@
> >     +----------------------------------------------------------------------+
> >   */
> > 
> > -/* $Id: streams.c,v 1.78 2005/05/16 08:37:13 tony2001 Exp $ */
> > +/* $Id: streams.c,v 1.79 2005/05/23 11:51:53 tony2001 Exp $ */
> > 
> >  #define _GNU_SOURCE
> >  #include "php.h"
> > @@ -603,8 +603,9 @@
> >                 }
> > 
> >                 /* just break anyway, to avoid greedy read */
> > -               if (stream->wrapper != &php_plain_files_wrapper)
> > +               if (stream->wrapper != NULL && stream->wrapper != 
> > &php_plain_files_wrapper) {
> >                         break;
> > +               }
> >         }
> > 
> >         if (didread > 0) {
> > 
> > --
> > PHP CVS Mailing List (http://www.php.net/)
> > To unsubscribe, visit: http://www.php.net/unsub.php
> > 
> >


-- 
Wbr, 
Antony Dovgal aka tony2001
[EMAIL PROTECTED]

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

Reply via email to