staging: dgap: fix checkpatch warning on sscanf usage
authorMark Hounschell <markh@compro.net>
Tue, 4 Mar 2014 21:03:10 +0000 (16:03 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 6 Mar 2014 22:14:37 +0000 (14:14 -0800)
This patch fixes a checkpatch warning for not checking
the return value of sscanf.

Signed-off-by: Mark Hounschell <markh@compro.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/dgap/dgap.c

index 6bd21e97ed8c6425327be00e0b46ba43643607f4..51f9924eac10b5606461696147b4a85d40a9edb6 100644 (file)
@@ -6017,7 +6017,8 @@ static ssize_t dgap_driver_pollrate_show(struct device_driver *ddp, char *buf)
 
 static ssize_t dgap_driver_pollrate_store(struct device_driver *ddp, const char *buf, size_t count)
 {
-       sscanf(buf, "%d\n", &dgap_poll_tick);
+       if (sscanf(buf, "%d\n", &dgap_poll_tick) != 1)
+               return -EINVAL;
        return count;
 }
 static DRIVER_ATTR(pollrate, (S_IRUSR | S_IWUSR), dgap_driver_pollrate_show, dgap_driver_pollrate_store);