diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2014-08-22 11:47:38 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2014-08-25 09:54:28 +1000 |
commit | 6247e5c12a6b1df5a0a68ea1beeddb48b9f7a495 (patch) | |
tree | f5f0a0467185f5bdfd7f8a2c78e355518f352b2d | |
parent | 400e52c6a285c214bf97a016aa94f5c6f68542bf (diff) | |
download | libevdev-6247e5c12a6b1df5a0a68ea1beeddb48b9f7a495.tar.gz |
uinput: explicitly ignore the UI_DEV_DESTROY return value
This can't fail in the kernel anyway, so cast it to shut up Coverity.
Error message:
"Calling function "ioctl(int, unsigned long, ...)" without checking return
value (as is done elsewhere 35 out of 36 times)."
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
-rw-r--r-- | libevdev/libevdev-uinput.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libevdev/libevdev-uinput.c b/libevdev/libevdev-uinput.c index ea2b025..33b5317 100644 --- a/libevdev/libevdev-uinput.c +++ b/libevdev/libevdev-uinput.c @@ -358,7 +358,7 @@ libevdev_uinput_destroy(struct libevdev_uinput *uinput_dev) if (!uinput_dev) return; - ioctl(uinput_dev->fd, UI_DEV_DESTROY, NULL); + (void)ioctl(uinput_dev->fd, UI_DEV_DESTROY, NULL); if (uinput_dev->fd_is_managed) close(uinput_dev->fd); free(uinput_dev->syspath); |