diff options
| author | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2013-09-11 21:56:58 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-10-01 09:17:46 -0700 |
| commit | 9f3383881d9b8f2ea3a1e8453e4e62a57b4af9ab (patch) | |
| tree | f409e72443db68eed64a1b8c2ea709967088e568 /drivers/hid/hid-lg2ff.c | |
| parent | ae5c98a48258dbdb742be23eadf9261b70d642c3 (diff) | |
HID: multitouch: validate indexes details
commit 8821f5dc187bdf16cfb32ef5aa8c3035273fa79a upstream.
When working on report indexes, always validate that they are in bounds.
Without this, a HID device could report a malicious feature report that
could trick the driver into a heap overflow:
[ 634.885003] usb 1-1: New USB device found, idVendor=0596, idProduct=0500
...
[ 676.469629] BUG kmalloc-192 (Tainted: G W ): Redzone overwritten
Note that we need to change the indexes from s8 to s16 as they can
be between -1 and 255.
CVE-2013-2897
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Acked-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/hid/hid-lg2ff.c')
0 files changed, 0 insertions, 0 deletions