Hello.
On 08/07/2014 10:22 AM, Lars R. Damerow wrote:
Replace a single-value sscanf() call with a call to kstrtou32(), as
recommended by checkpatch.pl.
Signed-off-by: Lars R. Damerow <l...@grandstreet.us>
---
drivers/staging/usbip/vhci_sysfs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/usbip/vhci_sysfs.c
b/drivers/staging/usbip/vhci_sysfs.c
index 211f43f..1dfbfa0 100644
--- a/drivers/staging/usbip/vhci_sysfs.c
+++ b/drivers/staging/usbip/vhci_sysfs.c
@@ -114,7 +114,7 @@ static ssize_t store_detach(struct device *dev, struct
device_attribute *attr,
int err;
__u32 rhport = 0;
- if (sscanf(buf, "%u", &rhport) != 1)
+ if (kstrtou32(buf, 10, &rhport) != 1)
Looks like you didn't bother studying what result this function returns...
hint: it returns 0 or negative value.
WBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
