[PATCH] Extract LogFinishInit from LogInit

Cyril Brulebois kibi at debian.org
Mon Nov 8 14:37:57 PST 2010


Mikhail Gusarov <dottedmag at dottedmag.net> (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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101108/8e48c16e/attachment.pgp>


More information about the xorg-devel mailing list