]>
Commit | Line | Data |
---|---|---|
0507c325 GKH |
1 | From dc50ddcd4c58a5a0226038307d6ef884bec9f8c2 Mon Sep 17 00:00:00 2001 |
2 | From: Stephane Grosjean <s.grosjean@peak-system.com> | |
3 | Date: Fri, 28 Nov 2014 14:08:48 +0100 | |
4 | Subject: can: peak_usb: fix memset() usage | |
5 | ||
6 | From: Stephane Grosjean <s.grosjean@peak-system.com> | |
7 | ||
8 | commit dc50ddcd4c58a5a0226038307d6ef884bec9f8c2 upstream. | |
9 | ||
10 | This patchs fixes a misplaced call to memset() that fills the request | |
11 | buffer with 0. The problem was with sending PCAN_USBPRO_REQ_FCT | |
12 | requests, the content set by the caller was thus lost. | |
13 | ||
14 | With this patch, the memory area is zeroed only when requesting info | |
15 | from the device. | |
16 | ||
17 | Signed-off-by: Stephane Grosjean <s.grosjean@peak-system.com> | |
18 | Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de> | |
19 | Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> | |
20 | ||
21 | --- | |
22 | drivers/net/can/usb/peak_usb/pcan_usb_pro.c | 3 +-- | |
23 | 1 file changed, 1 insertion(+), 2 deletions(-) | |
24 | ||
25 | --- a/drivers/net/can/usb/peak_usb/pcan_usb_pro.c | |
26 | +++ b/drivers/net/can/usb/peak_usb/pcan_usb_pro.c | |
27 | @@ -333,8 +333,6 @@ static int pcan_usb_pro_send_req(struct | |
28 | if (!(dev->state & PCAN_USB_STATE_CONNECTED)) | |
29 | return 0; | |
30 | ||
31 | - memset(req_addr, '\0', req_size); | |
32 | - | |
33 | req_type = USB_TYPE_VENDOR | USB_RECIP_OTHER; | |
34 | ||
35 | switch (req_id) { | |
36 | @@ -345,6 +343,7 @@ static int pcan_usb_pro_send_req(struct | |
37 | default: | |
38 | p = usb_rcvctrlpipe(dev->udev, 0); | |
39 | req_type |= USB_DIR_IN; | |
40 | + memset(req_addr, '\0', req_size); | |
41 | break; | |
42 | } | |
43 |