diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2015-05-06 09:20:44 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2015-05-08 14:08:58 +1000 |
commit | 217ab899cf4f14945d794d559629766cee206486 (patch) | |
tree | 5575c2a43abb95b96887ea49285273ca8a99cca6 | |
parent | 893fdf9b831772defdbd6ca34cd983a0975e0709 (diff) |
timer: fix coverity warning about unused return value
"read(int, void *, size_t)" returns the number of bytes read, but it
is ignored.
We don't really care about the number of bytes, but let's complain if we get
anything but EAGAIN.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
-rw-r--r-- | src/timer.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/timer.c b/src/timer.c index 114a6496..d1d3c108 100644 --- a/src/timer.c +++ b/src/timer.c @@ -102,8 +102,14 @@ libinput_timer_handler(void *data) struct libinput_timer *timer, *tmp; uint64_t now; uint64_t discard; + int r; - read(libinput->timer.fd, &discard, sizeof(discard)); + r = read(libinput->timer.fd, &discard, sizeof(discard)); + if (r == -1 && errno != EAGAIN) + log_bug_libinput(libinput, + "Error %d reading from timerfd (%s)", + errno, + strerror(errno)); now = libinput_now(libinput); if (now == 0) |