diff options
author | Jonathan Cameron <jic23@kernel.org> | 2012-08-27 17:40:04 (GMT) |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2012-08-27 17:58:36 (GMT) |
commit | 08ce9b44b53c580987b6a63df4e2206e45e20b92 (patch) | |
tree | 62e59a11a0048b3f26af6c616f7ab931aed97734 /drivers/iio/kfifo_buf.c | |
parent | c559afbfb08c7eac215ba417251225d3a8e01062 (diff) | |
download | linux-fsl-qoriq-08ce9b44b53c580987b6a63df4e2206e45e20b92.tar.xz |
iio:kfifo_buf improve error handling in read_first_n.
These two elements were originally in the patch
iio:kfifo_buf Take advantage of the fixed record size used in IIO
but Lars-Peter Clausen pointed out they should not have been
so here they are.
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Tested-by: Lars-Peter Clausen <lars@metafoo.de>
Diffstat (limited to 'drivers/iio/kfifo_buf.c')
-rw-r--r-- | drivers/iio/kfifo_buf.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/iio/kfifo_buf.c b/drivers/iio/kfifo_buf.c index 8a6d28c..6ec763f 100644 --- a/drivers/iio/kfifo_buf.c +++ b/drivers/iio/kfifo_buf.c @@ -108,10 +108,12 @@ static int iio_read_first_n_kfifo(struct iio_buffer *r, int ret, copied; struct iio_kfifo *kf = iio_to_kfifo(r); - if (n < r->bytes_per_datum) + if (n < r->bytes_per_datum || r->bytes_per_datum == 0) return -EINVAL; ret = kfifo_to_user(&kf->kf, buf, n, &copied); + if (ret < 0) + return ret; return copied; } |