[PATCH v3 2/3] config: Introduce InputAttributes in NewInputDeviceRequest

Adam Jackson ajax at nwnk.net
Tue Dec 15 07:08:24 PST 2009


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
Url : http://lists.x.org/archives/xorg-devel/attachments/20091215/9678e344/attachment.pgp 


More information about the xorg-devel mailing list