Mikhail Gusarov <[email protected]> (08/11/2010):
>  >> +    if (saveBuffer) {
>  >> +        free(saveBuffer);        /* Must be free(), not free() */
>  CB>                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> 
>  CB> That probably can go away, see 3f3ff971.
> 
> Right, as well as if (saveBuffer).

That reminded me I wanted to give Coccinelle a try. Just sent two
patches trying to fix most obvious if(foo[!=NULL]) free(foo) cases.

Mraw,
KiBi.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to