| From f71e41e23e129640f620b65fc362a6da02580310 Mon Sep 17 00:00:00 2001 |
| From: Tom Rix <trix@redhat.com> |
| Date: Sun, 9 Aug 2020 10:55:51 -0700 |
| Subject: iio:imu:st_lsm6dsx: check st_lsm6dsx_shub_read_output return |
| |
| From: Tom Rix <trix@redhat.com> |
| |
| commit f71e41e23e129640f620b65fc362a6da02580310 upstream. |
| |
| Potential error return is not checked. This can lead to use |
| of undefined data. |
| |
| Detected by clang static analysis. |
| |
| st_lsm6dsx_shub.c:540:8: warning: Assigned value is garbage or undefined |
| *val = (s16)le16_to_cpu(*((__le16 *)data)); |
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ |
| Fixes: c91c1c844ebd ("iio: imu: st_lsm6dsx: add i2c embedded controller support") |
| Signed-off-by: Tom Rix <trix@redhat.com |
| Cc: <Stable@vger.kernel.org> |
| Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com> |
| Link: https://lore.kernel.org/r/20200809175551.6794-1-trix@redhat.com |
| Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_shub.c | 2 ++ |
| 1 file changed, 2 insertions(+) |
| |
| --- a/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_shub.c |
| +++ b/drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_shub.c |
| @@ -313,6 +313,8 @@ st_lsm6dsx_shub_read(struct st_lsm6dsx_s |
| |
| err = st_lsm6dsx_shub_read_output(hw, data, |
| len & ST_LS6DSX_READ_OP_MASK); |
| + if (err < 0) |
| + return err; |
| |
| st_lsm6dsx_shub_master_enable(sensor, false); |
| |