Re: cvs: non-pecl /simple_cvs simple_cvs.c
| From: | Derick Rethans | Date: | Mon, 19 Apr 2004 11:11:41 +0000 |
| Subject: | Re: cvs: non-pecl /simple_cvs simple_cvs.c | ||
| References: | 1 2 3 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-27105@lists.php.net to get a copy of this message | ||
On Mon, 19 Apr 2004, Andi Gutmans wrote:
> At 12:42 PM 4/19/2004 +0200, Derick Rethans wrote:
> >On Mon, 19 Apr 2004, Shie Erlich wrote:
> >
> > > shie Mon Apr 19 06:35:07 2004 EDT
> > >
> > > Modified files:
> > > /non-pecl/simple_cvs simple_cvs.c
> > > Log:
> > > fixed: free(NULL) doesn't behave as the manpage says it does
> >
> >free(NULL) (c library function) is a no-op, but efree(NULL) (zend
> >memory alloc thingy) will crash.
>
> Yeah. I think we shouldn't add that additional if() to efree() and prefer
> to leave it as-is.
Right, it helps debugging things.
Derick