diff options
author | Hartmut Knaack <knaack.h@gmx.de> | 2015-05-31 08:40:05 -0400 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2015-05-31 14:25:12 -0400 |
commit | 963f54cef23b7e8c91bbe60b978b5f4a3e990f2c (patch) | |
tree | 34b452ec0d0695fd41b269e18c0850ca0ff7b474 /tools/iio | |
parent | 6bb7cac8551e2d60edbd25a6d046cc45932c4c3e (diff) |
tools:iio:iio_event_monitor: add error handling
Add error handling to calls which can indicate a major problem by
returning an error code.
Signed-off-by: Hartmut Knaack <knaack.h@gmx.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'tools/iio')
-rw-r--r-- | tools/iio/iio_event_monitor.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/tools/iio/iio_event_monitor.c b/tools/iio/iio_event_monitor.c index 1374374208ee..1316527f3c3c 100644 --- a/tools/iio/iio_event_monitor.c +++ b/tools/iio/iio_event_monitor.c | |||
@@ -286,7 +286,10 @@ int main(int argc, char **argv) | |||
286 | goto error_free_chrdev_name; | 286 | goto error_free_chrdev_name; |
287 | } | 287 | } |
288 | 288 | ||
289 | close(fd); | 289 | if (close(fd) == -1) { |
290 | ret = -errno; | ||
291 | goto error_free_chrdev_name; | ||
292 | } | ||
290 | 293 | ||
291 | while (true) { | 294 | while (true) { |
292 | ret = read(event_fd, &event, sizeof(event)); | 295 | ret = read(event_fd, &event, sizeof(event)); |
@@ -304,7 +307,9 @@ int main(int argc, char **argv) | |||
304 | print_event(&event); | 307 | print_event(&event); |
305 | } | 308 | } |
306 | 309 | ||
307 | close(event_fd); | 310 | if (close(event_fd) == -1) |
311 | perror("Failed to close event file"); | ||
312 | |||
308 | error_free_chrdev_name: | 313 | error_free_chrdev_name: |
309 | free(chrdev_name); | 314 | free(chrdev_name); |
310 | error_ret: | 315 | error_ret: |