Warn if there are multiple devices with the same syspath
authorPeter Hutterer <peter.hutterer@who-t.net>
Sat, 31 Aug 2013 03:03:47 +0000 (13:03 +1000)
committerPeter Hutterer <peter.hutterer@who-t.net>
Tue, 3 Sep 2013 07:01:42 +0000 (17:01 +1000)
Change to the previous code in that we continue looking at devices
even after we've found one. However, this way we can warn
the user when we can't guarantee syspath correctness.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
libevdev/libevdev-uinput.c

index afebc0b..af98fa4 100644 (file)
@@ -226,11 +226,16 @@ fetch_syspath_and_devnode(struct libevdev_uinput *uinput_dev)
 
                buf[len - 1] = '\0'; /* file contains \n */
                if (strcmp(buf, uinput_dev->name) == 0) {
-                       strcpy(buf, SYS_INPUT_DIR);
-                       strcat(buf, namelist[i]->d_name);
-                       uinput_dev->syspath = strdup(buf);
-                       uinput_dev->devnode = fetch_device_node(buf);
-                       break;
+                       if (uinput_dev->syspath) {
+                               /* FIXME: could descend into bit comparison here */
+                               log_info("multiple identical devices found. syspath is unreliable\n");
+                               break;
+                       } else {
+                               strcpy(buf, SYS_INPUT_DIR);
+                               strcat(buf, namelist[i]->d_name);
+                               uinput_dev->syspath = strdup(buf);
+                               uinput_dev->devnode = fetch_device_node(buf);
+                       }
                }
        }