The semantic patch that makes this change is available
in scripts/coccinelle/api/memdup.cocci.

Signed-off-by: Thomas Meyer <[email protected]>
---

diff -u -p a/drivers/media/video/pwc/pwc-ctrl.c 
b/drivers/media/video/pwc/pwc-ctrl.c
--- a/drivers/media/video/pwc/pwc-ctrl.c 2011-11-07 19:37:51.143333699 +0100
+++ b/drivers/media/video/pwc/pwc-ctrl.c 2011-11-08 10:47:00.679677247 +0100
@@ -113,10 +113,9 @@ static int _send_control_msg(struct pwc_
        void *kbuf = NULL;
 
        if (buflen) {
-               kbuf = kmalloc(buflen, GFP_KERNEL); /* not allowed on stack */
+               kbuf = kmemdup(buf, buflen, GFP_KERNEL); /* not allowed on 
stack */
                if (kbuf == NULL)
                        return -ENOMEM;
-               memcpy(kbuf, buf, buflen);
        }
 
        rc = usb_control_msg(pdev->udev, usb_sndctrlpipe(pdev->udev, 0),
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to