On 01/31/11 04:02 AM, Erkki Seppälä wrote:
> From: Ander Conselvan de Oliveira <[email protected]>
>
> xcms/LRGB: Add a label for freeing property_return.
That's a much better subject/shortlog line than the far too long one you have
there now.
> @@ -789,8 +782,6 @@ LINEAR_RGB_InitSCCData(
> #endif /* ALLDEBUG */
> }
>
> - Xfree ((char *)property_return);
> -
> /* Free the old memory and use the new structure created. */
> LINEAR_RGB_FreeSCCData(pPerScrnInfo->screenData);
Don't you need to leave that one there since you return(XcmsSuccess);
in that case, without going down through the Free* / return(XcmsFailure) path?
--
-Alan Coopersmith- [email protected]
Oracle Solaris Platform Engineering: X Window System
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
