From 37a389ae91c05a82c58cd54f8a8f3d70c47a9f2b Mon Sep 17 00:00:00 2001 From: Vijayavardhan Vennapusa Date: Wed, 13 Sep 2017 15:41:20 +0530 Subject: USB: composite: Use snprintf instead of sprintf composite driver is using sprintf() function to expose value to userspace. This sprintf() is banned function and instead required to use snprintf() function. Hence replace sprintf() with snprintf() function to fix static analysis tool error. Change-Id: I85f38fe3f2121f6a0cb76b7178a3e699ba1dc7f2 Signed-off-by: Vijayavardhan Vennapusa --- drivers/usb/gadget/composite.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c index 442d44278f33..22ba45f40f0b 100644 --- a/drivers/usb/gadget/composite.c +++ b/drivers/usb/gadget/composite.c @@ -2088,7 +2088,7 @@ static ssize_t suspended_show(struct device *dev, struct device_attribute *attr, struct usb_gadget *gadget = dev_to_usb_gadget(dev); struct usb_composite_dev *cdev = get_gadget_data(gadget); - return sprintf(buf, "%d\n", cdev->suspended); + return snprintf(buf, PAGE_SIZE, "%d\n", cdev->suspended); } static DEVICE_ATTR_RO(suspended); -- cgit v1.2.3