HID: zeroplus: validate output report details
authorKees Cook <keescook@chromium.org>
Wed, 11 Sep 2013 19:56:51 +0000 (21:56 +0200)
committerBen Hutchings <ben@decadent.org.uk>
Sat, 26 Oct 2013 20:06:04 +0000 (21:06 +0100)
commite0f897f0929d27dc5d7de5cc24ccbeac572e5176
tree8eb4ed9c2c00b2e09dbcde082cf1c4c71327b5a6
parent9c8ea9f2d66ff437dd25da392a104947c703d975
HID: zeroplus: validate output report details

commit 78214e81a1bf43740ce89bb5efda78eac2f8ef83 upstream.

The zeroplus HID driver was not checking the size of allocated values
in fields it used. A HID device could send a malicious output report
that would cause the driver to write beyond the output report allocation
during initialization, causing a heap overflow:

[ 1442.728680] usb 1-1: New USB device found, idVendor=0c12, idProduct=0005
...
[ 1466.243173] BUG kmalloc-192 (Tainted: G        W   ): Redzone overwritten

CVE-2013-2889

Signed-off-by: Kees Cook <keescook@chromium.org>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/hid/hid-zpff.c