diff options
author | Ioana Ciornei <ciorneiioana@gmail.com> | 2015-11-01 07:58:45 -0500 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2015-11-08 10:44:10 -0500 |
commit | 72a868b38bdd60cbc4084a91fd7b8df3e2bb96ba (patch) | |
tree | 8e7ff9ec5d6ce43c24005d648fd82f04ad970c78 | |
parent | a106b4748917ba510d083217dbc25e56299f32d4 (diff) |
iio: imu: check sscanf return value
This patch fixes the following checkpatch warning:
WARNING: unchecked sscanf return value
Signed-off-by: Ioana Ciornei <ciorneiioana@gmail.com>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
-rw-r--r-- | drivers/iio/imu/adis16400_core.c | 6 | ||||
-rw-r--r-- | drivers/iio/imu/adis16480.c | 4 |
2 files changed, 8 insertions, 2 deletions
diff --git a/drivers/iio/imu/adis16400_core.c b/drivers/iio/imu/adis16400_core.c index abc4c50de9e8..72bcc2491d1d 100644 --- a/drivers/iio/imu/adis16400_core.c +++ b/drivers/iio/imu/adis16400_core.c | |||
@@ -288,7 +288,11 @@ static int adis16400_initial_setup(struct iio_dev *indio_dev) | |||
288 | if (ret) | 288 | if (ret) |
289 | goto err_ret; | 289 | goto err_ret; |
290 | 290 | ||
291 | sscanf(indio_dev->name, "adis%u\n", &device_id); | 291 | ret = sscanf(indio_dev->name, "adis%u\n", &device_id); |
292 | if (ret != 1) { | ||
293 | ret = -EINVAL; | ||
294 | goto err_ret; | ||
295 | } | ||
292 | 296 | ||
293 | if (prod_id != device_id) | 297 | if (prod_id != device_id) |
294 | dev_warn(&indio_dev->dev, "Device ID(%u) and product ID(%u) do not match.", | 298 | dev_warn(&indio_dev->dev, "Device ID(%u) and product ID(%u) do not match.", |
diff --git a/drivers/iio/imu/adis16480.c b/drivers/iio/imu/adis16480.c index b94bfd3f595b..16d430461414 100644 --- a/drivers/iio/imu/adis16480.c +++ b/drivers/iio/imu/adis16480.c | |||
@@ -765,7 +765,9 @@ static int adis16480_initial_setup(struct iio_dev *indio_dev) | |||
765 | if (ret) | 765 | if (ret) |
766 | return ret; | 766 | return ret; |
767 | 767 | ||
768 | sscanf(indio_dev->name, "adis%u\n", &device_id); | 768 | ret = sscanf(indio_dev->name, "adis%u\n", &device_id); |
769 | if (ret != 1) | ||
770 | return -EINVAL; | ||
769 | 771 | ||
770 | if (prod_id != device_id) | 772 | if (prod_id != device_id) |
771 | dev_warn(&indio_dev->dev, "Device ID(%u) and product ID(%u) do not match.", | 773 | dev_warn(&indio_dev->dev, "Device ID(%u) and product ID(%u) do not match.", |