diff options
author | Muhammad Falak R Wani <falakreyaz@gmail.com> | 2016-05-19 10:04:36 -0400 |
---|---|---|
committer | Johan Hovold <johan@kernel.org> | 2016-07-15 08:23:21 -0400 |
commit | d6c4dc3bf4e934fae7002f296aaef4b578f7ca14 (patch) | |
tree | bbd9f52d4c241faff29e296b12faf2fbf040b8d8 | |
parent | d8d841e8332779fae2b18420d39ef407ea3729da (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.c | 4 |
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, |