[PATCH] Fix gensupport.c memory leak (take 3)

Andrew Pinski pinskia@physics.uc.edu
Fri Mar 17 00:17:00 GMT 2006


On Mar 16, 2006, at 6:58 PM, Roger Sayle wrote:

>
> Hi Andrew,
>
> On Thu, 16 Mar 2006, Andrew Pinski wrote:
>> You really don't need the if here as free handles NULL pointers just
>> fine.
>
> The "if" isn't required for correct operation on C99 targets,

Actually any C90/C89 targets.  Since we already require a C90 compiler,
shouldn't this be a no brainer?  Also we already do realloc all the time
on a NULL pointer without any bugs being filed :).

-- Pinski



More information about the Gcc-patches mailing list