[PATCH 1/2] [libX11] xcms/LRGB: don't double-free property_return

2011-02-04 Thread Erkki Seppälä
From: Ander Conselvan de Oliveira property_return was free'd before and in the case the conditional is true, the call to XcmsGetProperty failed which means that property_return wasn't set so there is no need to free it again. Double free of pointer "property_return" in call to "free" Reviewed-b

Re: [PATCH 1/2] [libX11] xcms/LRGB: don't double-free property_return

2011-02-09 Thread Alan Coopersmith
On 02/ 4/11 01:43 AM, Erkki Seppälä wrote: > From: Ander Conselvan de Oliveira > > property_return was free'd before and in the case the conditional is > true, the call to XcmsGetProperty failed which means that > property_return wasn't set so there is no need to free it again. > > Double free o