From: Ander Conselvan de Oliveira <ander.conselvan-de-olive...@nokia.com>
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. Reviewed-by: Erkki Seppälä <erkki.sepp...@vincit.fi> Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan-de-olive...@nokia.com> --- src/xcms/LRGB.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/xcms/LRGB.c b/src/xcms/LRGB.c index 2f7a4cc..d9168bd 100644 --- a/src/xcms/LRGB.c +++ b/src/xcms/LRGB.c @@ -573,7 +573,6 @@ LINEAR_RGB_InitSCCData( if (CorrectAtom == None || !_XcmsGetProperty (dpy, RootWindow(dpy, screenNumber), CorrectAtom, &format_return, &nitems, &nbytes_return, &property_return)) { - Xfree ((char *)property_return); goto FreeSCCData; } -- 1.7.0.4 _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel