Techwizz/kernel-fxtec-pro1x
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Staging: iio: Documentation: Added check for sscanf return value.

Browse source 
The following patch fixes the following checkpatch.pl warning by adding
a check for sscanf return value.
WARNING: unchecked sscanf return value.

Signed-off-by: Heena Sirwani <heenasirwani@gmail.com>
Acked-by: Daniel Baluta <daniel.baluta@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Heena Sirwani 2014-10-06 17:04:43 +05:30 committed by Greg Kroah-Hartman
parent edead9b1f9
commit d0e68ce199
1 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
drivers/staging/iio/Documentation/lsiio.c
View file

@ -63,9 +63,12 @@ static int dump_one_device(const char *dev_dir_name)
{ {
char name[IIO_MAX_NAME_LENGTH]; char name[IIO_MAX_NAME_LENGTH];
int dev_idx; int dev_idx;
int retval;
sscanf(dev_dir_name + strlen(iio_dir) + strlen(type_device), retval = sscanf(dev_dir_name + strlen(iio_dir) + strlen(type_device),
"%i", &dev_idx); "%i", &dev_idx);
if (retval != 1)
return -EINVAL;
read_sysfs_string("name", dev_dir_name, name); read_sysfs_string("name", dev_dir_name, name);
printf("Device %03d: %s\n", dev_idx, name); printf("Device %03d: %s\n", dev_idx, name);
@ -78,9 +81,12 @@ static int dump_one_trigger(const char *dev_dir_name)
{ {
char name[IIO_MAX_NAME_LENGTH]; char name[IIO_MAX_NAME_LENGTH];
int dev_idx; int dev_idx;
int retval;
sscanf(dev_dir_name + strlen(iio_dir) + strlen(type_trigger), retval = sscanf(dev_dir_name + strlen(iio_dir) + strlen(type_trigger),
"%i", &dev_idx); "%i", &dev_idx);
if (retval != 1)
return -EINVAL;
read_sysfs_string("name", dev_dir_name, name); read_sysfs_string("name", dev_dir_name, name);
printf("Trigger %03d: %s\n", dev_idx, name); printf("Trigger %03d: %s\n", dev_idx, name);
return 0; return 0;