aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIoana Ciornei <ciorneiioana@gmail.com>2015-11-01 07:58:45 -0500
committerJonathan Cameron <jic23@kernel.org>2015-11-08 10:44:10 -0500
commit72a868b38bdd60cbc4084a91fd7b8df3e2bb96ba (patch)
tree8e7ff9ec5d6ce43c24005d648fd82f04ad970c78
parenta106b4748917ba510d083217dbc25e56299f32d4 (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.c6
-rw-r--r--drivers/iio/imu/adis16480.c4
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.",