On Mon, 2009-12-14 at 21:09 -0800, Dan Nicholson wrote:

> @@ -430,6 +449,13 @@ unwind:
>          xfree(tmpo);
>      }
>  
> +    if (attrs.product)
> +        xfree(attrs.product);
> +    if (attrs.vendor)
> +        xfree(attrs.vendor);
> +    if (attrs.device)
> +        xfree(attrs.device);
> +
>      if (xkb_opts.layout)
>          xfree(xkb_opts.layout);
>      if (xkb_opts.rules)

xfree(NULL), like free(NULL), is a legal no-op.

I like the rest of this series in concept at least, very cool.

- ajax

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
xorg-devel mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to