[PATCH] Remove superfluous if(p!=NULL) checks around free(p); p=NULL;

Matt Turner mattst88 at gmail.com
Wed Nov 10 08:05:10 PST 2010


On Wed, Nov 10, 2010 at 10:57 AM, Jonathan Morton
<jonathan.morton at movial.com> wrote:
> On Wed, 2010-11-10 at 10:57 -0500, Matt Turner wrote:
>> I'm not so sure about this. Do you know for sure that nothing depends
>> on setting the value of the freed pointer to NULL or 0 afterwards?
>
> If the pointer is already NULL, surely setting it to NULL again is not a
> problem?
>
> --
> ------
> From: Jonathan Morton
>      jonathan.morton at movial.com

Sigh. I somehow managed to interpret this as we were removing the p =
NULL lines.

Please ignore me.


More information about the xorg-devel mailing list