summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>2012-01-24 13:43:21 +0200
committerLaurent Pinchart <laurent.pinchart@ideasonboard.com>2012-01-25 15:47:41 +0100
commit42ad18e4d639f17f3e57ba4ed330cd555605b178 (patch)
tree99a6bf6060542d84484eded1fae026912d805f27
parente6062dc3643b64cb009c12edef1395d2b5504f4a (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.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/yavta.c b/yavta.c
index b5b4487..2636577 100644
--- a/yavta.c
+++ b/yavta.c
@@ -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,