[PATCH:xdm 1/5] Fix potential memory leak in allocation failure cleanup path

2011-05-14 Thread Alan Coopersmith
Error: Memory leak (CWE 401) Memory leak of pointer 'newargv' allocated with realloc(((i8*)argv), ((i * 4) + 8)) at line 239 of /export/alanc/X.Org/git/app/xdm/xdm/util.c in function 'parseArgs'. 'newargv' allocated at line 231 with realloc(((i8*)argv), ((i * 4) + 8)).

Re: [PATCH:xdm 1/5] Fix potential memory leak in allocation failure cleanup path

2011-05-14 Thread Mikhail Gusarov
Twas brillig at 11:01:29 14.05.2011 UTC-07 when alan.coopersm...@oracle.com did gyre and gimble: AC> Signed-off-by: Alan Coopersmith Reviewed-by: Mikhail Gusarov for the series. -- http://fossarchy.blogspot.com/ pgp8Z3uXrP46I.pgp Description: PGP signature __

Re: [PATCH:xdm 1/5] Fix potential memory leak in allocation failure cleanup path

2011-05-14 Thread Julien Cristau
On Sat, May 14, 2011 at 11:01:29 -0700, Alan Coopersmith wrote: > Error: Memory leak (CWE 401) >Memory leak of pointer 'newargv' allocated with realloc(((i8*)argv), ((i * > 4) + 8)) > at line 239 of /export/alanc/X.Org/git/app/xdm/xdm/util.c in function > 'parseArgs'. > 'ne