aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMuhammad Falak R Wani <falakreyaz@gmail.com>2016-05-19 10:04:36 -0400
committerJohan Hovold <johan@kernel.org>2016-07-15 08:23:21 -0400
commitd6c4dc3bf4e934fae7002f296aaef4b578f7ca14 (patch)
treebbd9f52d4c241faff29e296b12faf2fbf040b8d8
parentd8d841e8332779fae2b18420d39ef407ea3729da (diff)
USB: serial: cp210x: use kmemdup
Use kmemdup when some other buffer is immediately copied into allocated region. It replaces call to allocation followed by memcpy, by a single call to kmemdup. Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com> Signed-off-by: Johan Hovold <johan@kernel.org>
-rw-r--r--drivers/usb/serial/cp210x.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/usb/serial/cp210x.c b/drivers/usb/serial/cp210x.c
index 96a70789b4c2..4d6a5c672a3d 100644
--- a/drivers/usb/serial/cp210x.c
+++ b/drivers/usb/serial/cp210x.c
@@ -496,12 +496,10 @@ static int cp210x_write_reg_block(struct usb_serial_port *port, u8 req,
496 void *dmabuf; 496 void *dmabuf;
497 int result; 497 int result;
498 498
499 dmabuf = kmalloc(bufsize, GFP_KERNEL); 499 dmabuf = kmemdup(buf, bufsize, GFP_KERNEL);
500 if (!dmabuf) 500 if (!dmabuf)
501 return -ENOMEM; 501 return -ENOMEM;
502 502
503 memcpy(dmabuf, buf, bufsize);
504
505 result = usb_control_msg(serial->dev, usb_sndctrlpipe(serial->dev, 0), 503 result = usb_control_msg(serial->dev, usb_sndctrlpipe(serial->dev, 0),
506 req, REQTYPE_HOST_TO_INTERFACE, 0, 504 req, REQTYPE_HOST_TO_INTERFACE, 0,
507 port_priv->bInterfaceNumber, dmabuf, bufsize, 505 port_priv->bInterfaceNumber, dmabuf, bufsize,