diff options
author | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2013-09-11 21:56:57 +0200 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2013-09-13 15:13:22 +0200 |
commit | cc6b54aa54bf40b762cab45a9fc8aa81653146eb (patch) | |
tree | 311d5b5b18d0d59e2cda8b45146c8a6ab3cc5fee /drivers/hid/hid-logitech-dj.c | |
parent | HID: lenovo-tpkbd: validate output report details (diff) | |
download | linux-cc6b54aa54bf40b762cab45a9fc8aa81653146eb.tar.xz linux-cc6b54aa54bf40b762cab45a9fc8aa81653146eb.zip |
HID: validate feature and input report details
When dealing with usage_index, be sure to properly use unsigned instead of
int to avoid overflows.
When working on report fields, always validate that their report_counts 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
CVE-2013-2897
Cc: stable@vger.kernel.org
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Acked-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid/hid-logitech-dj.c')
0 files changed, 0 insertions, 0 deletions