diff options
author | Johan Hovold <jhovold@gmail.com> | 2009-12-28 17:01:56 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-03-02 17:53:56 -0500 |
commit | d2126326bd71b56fcaa5e86474433d11e253f84d (patch) | |
tree | 67fb9604d46b721c94e6246d55c0bfd3c6101e60 /drivers/usb | |
parent | 9e221a35f82cbef0397d81fed588bafba95b550c (diff) |
USB: oti6858: fix DMA buffer on stack
Signed-off-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/serial/oti6858.c | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/drivers/usb/serial/oti6858.c b/drivers/usb/serial/oti6858.c index c644e26394b4..2ae97e7195c6 100644 --- a/drivers/usb/serial/oti6858.c +++ b/drivers/usb/serial/oti6858.c | |||
@@ -302,7 +302,7 @@ void send_data(struct work_struct *work) | |||
302 | struct usb_serial_port *port = priv->port; | 302 | struct usb_serial_port *port = priv->port; |
303 | int count = 0, result; | 303 | int count = 0, result; |
304 | unsigned long flags; | 304 | unsigned long flags; |
305 | unsigned char allow; | 305 | u8 *allow; |
306 | 306 | ||
307 | dbg("%s(port = %d)", __func__, port->number); | 307 | dbg("%s(port = %d)", __func__, port->number); |
308 | 308 | ||
@@ -321,13 +321,20 @@ void send_data(struct work_struct *work) | |||
321 | count = port->bulk_out_size; | 321 | count = port->bulk_out_size; |
322 | 322 | ||
323 | if (count != 0) { | 323 | if (count != 0) { |
324 | allow = kmalloc(1, GFP_KERNEL); | ||
325 | if (!allow) { | ||
326 | dev_err(&port->dev, "%s(): kmalloc failed\n", | ||
327 | __func__); | ||
328 | return; | ||
329 | } | ||
324 | result = usb_control_msg(port->serial->dev, | 330 | result = usb_control_msg(port->serial->dev, |
325 | usb_rcvctrlpipe(port->serial->dev, 0), | 331 | usb_rcvctrlpipe(port->serial->dev, 0), |
326 | OTI6858_REQ_T_CHECK_TXBUFF, | 332 | OTI6858_REQ_T_CHECK_TXBUFF, |
327 | OTI6858_REQ_CHECK_TXBUFF, | 333 | OTI6858_REQ_CHECK_TXBUFF, |
328 | count, 0, &allow, 1, 100); | 334 | count, 0, allow, 1, 100); |
329 | if (result != 1 || allow != 0) | 335 | if (result != 1 || *allow != 0) |
330 | count = 0; | 336 | count = 0; |
337 | kfree(allow); | ||
331 | } | 338 | } |
332 | 339 | ||
333 | if (count == 0) { | 340 | if (count == 0) { |