[media] ati_remote: avoid fragile snprintf use
authorRasmus Villemoes <linux@rasmusvillemoes.dk>
Tue, 8 Mar 2016 20:40:53 +0000 (17:40 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Sat, 7 May 2016 13:27:16 +0000 (10:27 -0300)
Passing overlapping source and destination to snprintf is
fragile. Replace with a single (mostly) equivalent call. If one wants
to preserve the space preceding udev->product whether or not there was
a manufacturer, just remove udev->manufacturer from the && expression.

Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/rc/ati_remote.c

index 3f61d77..9f5b597 100644 (file)
@@ -873,13 +873,10 @@ static int ati_remote_probe(struct usb_interface *interface,
        strlcat(ati_remote->rc_phys, "/input0", sizeof(ati_remote->rc_phys));
        strlcat(ati_remote->mouse_phys, "/input1", sizeof(ati_remote->mouse_phys));
 
-       if (udev->manufacturer)
-               strlcpy(ati_remote->rc_name, udev->manufacturer,
-                       sizeof(ati_remote->rc_name));
-
-       if (udev->product)
-               snprintf(ati_remote->rc_name, sizeof(ati_remote->rc_name),
-                        "%s %s", ati_remote->rc_name, udev->product);
+       snprintf(ati_remote->rc_name, sizeof(ati_remote->rc_name), "%s%s%s",
+               udev->manufacturer ?: "",
+               udev->manufacturer && udev->product ? " " : "",
+               udev->product ?: "");
 
        if (!strlen(ati_remote->rc_name))
                snprintf(ati_remote->rc_name, sizeof(ati_remote->rc_name),