diff --git a/drivers/usb/gadget/configfs.c b/drivers/usb/gadget/configfs.c index a2b0cb2ac64a..88b921e04d49 100644 --- a/drivers/usb/gadget/configfs.c +++ b/drivers/usb/gadget/configfs.c @@ -140,28 +140,21 @@ struct gadget_config_name { struct list_head list; }; -#define MAX_USB_STRING_LEN 126 -#define MAX_USB_STRING_WITH_NULL_LEN (MAX_USB_STRING_LEN+1) - static int usb_string_copy(const char *s, char **s_copy) { int ret; char *str; char *copy = *s_copy; ret = strlen(s); - if (ret > MAX_USB_STRING_LEN) + if (ret > 126) return -EOVERFLOW; - if (copy) { - str = copy; - } else { - str = kmalloc(MAX_USB_STRING_WITH_NULL_LEN, GFP_KERNEL); - if (!str) - return -ENOMEM; - } - strncpy(str, s, MAX_USB_STRING_WITH_NULL_LEN); + str = kstrdup(s, GFP_KERNEL); + if (!str) + return -ENOMEM; if (str[ret - 1] == '\n') str[ret - 1] = '\0'; + kfree(copy); *s_copy = str; return 0; }