diff options
| author | Ezequiel Garcia <ezequiel.garcia@free-electrons.com> | 2014-04-17 09:28:20 -0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-06-30 20:11:58 -0700 |
| commit | 1bae99bdf42d4587b5cbe2ec255975abb83f882c (patch) | |
| tree | b269c2a3dcf5b6149b66f6b632dfdd8a17c7cc79 /include | |
| parent | edf9ebfc0e9cb7058d97cffadee22f5aa94cc19e (diff) | |
media: stk1160: Avoid stack-allocated buffer for control URBs
commit 85ac1a1772bb41da895bad83a81f6a62c8f293f6 upstream.
Currently stk1160_read_reg() uses a stack-allocated char to get the
read control value. This is wrong because usb_control_msg() requires
a kmalloc-ed buffer.
This commit fixes such issue by kmalloc'ating a 1-byte buffer to receive
the read value.
While here, let's remove the urb_buf array which was meant for a similar
purpose, but never really used.
Cc: Alan Stern <stern@rowland.harvard.edu>
Reported-by: Sander Eikelenboom <linux@eikelenboom.it>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions