iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

iio: addac: ad74413r: fix off by one in ad74413r_parse_channel_config()

Browse Source 
The > needs to be >= to prevent accessing one element beyond the end of
the st->channel_configs[] array.

Fixes: fea251b6a5 ("iio: addac: add AD74413R driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Cosmin Tanislav <cosmin.tanislav@analog.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
This commit is contained in:
Dan Carpenter 2021-12-15 14:50:53 +03:00 committed by Jonathan Cameron
parent 0a52c3f347
commit 5d97d9e9a7
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/iio/addac/ad74413r.c
View File

@ -1150,7 +1150,7 @@ static int ad74413r_parse_channel_config(struct iio_dev *indio_dev,
return ret; return ret;
} }
if (index > AD74413R_CHANNEL_MAX) { if (index >= AD74413R_CHANNEL_MAX) {
dev_err(st->dev, "Channel index %u is too large\n", index); dev_err(st->dev, "Channel index %u is too large\n", index);
return -EINVAL; return -EINVAL;
} }