diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2020-01-16 13:08:29 +0300 |
|---|---|---|
| committer |   Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2020-01-18 13:33:11 +0000 |
| commit | 98496ccdf0dd88696203e69542e07ef824c56ead (patch) | |
| tree | f6aa2016245b1dcd64f58ef787f7385e7ea24079 /drivers/iio | |
| parent | iio: imu/mpu6050: support dual-edge IRQ (diff) | |
| download | linux-dev-98496ccdf0dd88696203e69542e07ef824c56ead.tar.xz linux-dev-98496ccdf0dd88696203e69542e07ef824c56ead.zip | |
iio: accel: bma400: prevent setting accel scale too low
This puts an upper bound on "val2" but it also needs to have a lower
bound (BMA400_SCALE_MIN).
Fixes: 465c811f1f20 ("iio: accel: Add driver for the BMA400")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Diffstat (limited to 'drivers/iio')
| -rw-r--r-- | drivers/iio/accel/bma400_core.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/iio/accel/bma400_core.c b/drivers/iio/accel/bma400_core.c index ab4a158b35af..cc77f89c048b 100644 --- a/drivers/iio/accel/bma400_core.c +++ b/drivers/iio/accel/bma400_core.c @@ -752,7 +752,8 @@ static int bma400_write_raw(struct iio_dev *indio_dev, mutex_unlock(&data->mutex); return ret; case IIO_CHAN_INFO_SCALE: - if (val != 0 || val2 > BMA400_SCALE_MAX) + if (val != 0 || + val2 < BMA400_SCALE_MIN || val2 > BMA400_SCALE_MAX) return -EINVAL; mutex_lock(&data->mutex); |