diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2012-01-24 13:43:21 +0200 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2012-01-25 15:47:41 +0100 |
commit | 42ad18e4d639f17f3e57ba4ed330cd555605b178 (patch) | |
tree | 99a6bf6060542d84484eded1fae026912d805f27 | |
parent | e6062dc3643b64cb009c12edef1395d2b5504f4a (diff) |
warn if written size is different to original one
This patch also eliminates compiler warning:
yavta.c: In function ‘video_save_image’:
yavta.c:974:6: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
[Print different messages depending on whether the return value is
negative or positive]
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
-rw-r--r-- | yavta.c | 6 |
1 files changed, 6 insertions, 0 deletions
@@ -1006,6 +1006,12 @@ static void video_save_image(struct device *dev, struct v4l2_buffer *buf, ret = write(fd, dev->buffers[buf->index].mem, buf->bytesused); close(fd); + + if (ret < 0) + printf("write error: %s (%d)\n", strerror(errno), errno); + else if (ret != (int)buf->bytesused) + printf("write error: only %d bytes written instead of %u\n", + ret, buf->bytesused); } static int video_do_capture(struct device *dev, unsigned int nframes, |