diff options
| author | Peter Hutterer <peter.hutterer@who-t.net> | 2015-08-18 14:59:06 +1000 |
|---|---|---|
| committer | Peter Hutterer <peter.hutterer@who-t.net> | 2015-08-19 14:14:31 +1000 |
| commit | 004b167469e8701f5118b5cb5c98b23521fbfc94 (patch) | |
| tree | df384b305d5fafc9c599a257594d194c47248286 | |
| parent | 5203c693a5a1ea875a9196b281bf9d2407a49840 (diff) | |
tools: handle sscanf return value
Coverity complains. There isn't really a case where this should go wrong, so
we can silently ignore the error case.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
| -rw-r--r-- | tools/find_event_devices.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tools/find_event_devices.c b/tools/find_event_devices.c index 1f7539a..36aa3ac 100644 --- a/tools/find_event_devices.c +++ b/tools/find_event_devices.c @@ -47,6 +47,7 @@ char* find_event_devices(void) int i, ndev, devnum; char *filename; int max_device = 0; + int rc; ndev = scandir(DEV_INPUT_EVENT, &namelist, is_event_device, versionsort); if (ndev <= 0) @@ -70,8 +71,8 @@ char* find_event_devices(void) fprintf(stderr, "%s: %s\n", fname, name); close(fd); - sscanf(namelist[i]->d_name, "event%d", &devnum); - if (devnum > max_device) + rc = sscanf(namelist[i]->d_name, "event%d", &devnum); + if (rc == 1 && devnum > max_device) max_device = devnum; free(namelist[i]); |