usb: gadget: inode: switch over to memdup_user()
authorFelipe Balbi <balbi@ti.com>
Mon, 10 Mar 2014 18:30:55 +0000 (13:30 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 17 Mar 2014 23:05:31 +0000 (16:05 -0700)
This patch fixes the following Coccinelle warning:

drivers/usb/gadget/inode.c:442:8-15: WARNING \
opportunity for memdup_user

Signed-off-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/gadget/inode.c

index b94c049ab0d0877762a5d76e1cac957eff34771f..b5be6f0308c270f2a844db3f00e7884380e5c2e6 100644 (file)
@@ -439,11 +439,9 @@ ep_write (struct file *fd, const char __user *buf, size_t len, loff_t *ptr)
        /* FIXME writebehind for O_NONBLOCK and poll(), qlen = 1 */
 
        value = -ENOMEM;
-       kbuf = kmalloc (len, GFP_KERNEL);
-       if (!kbuf)
-               goto free1;
-       if (copy_from_user (kbuf, buf, len)) {
-               value = -EFAULT;
+       kbuf = memdup_user(buf, len);
+       if (!kbuf) {
+               value = PTR_ERR(kbuf);
                goto free1;
        }
 
@@ -452,7 +450,6 @@ ep_write (struct file *fd, const char __user *buf, size_t len, loff_t *ptr)
                data->name, len, (int) value);
 free1:
        mutex_unlock(&data->lock);
-       kfree (kbuf);
        return value;
 }