diff options
| author | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2013-09-11 19:56:57 (GMT) |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2013-09-13 13:13:22 (GMT) |
| commit | cc6b54aa54bf40b762cab45a9fc8aa81653146eb (patch) | |
| tree | 311d5b5b18d0d59e2cda8b45146c8a6ab3cc5fee /ipc | |
| parent | 0a9cd0a80ac559357c6a90d26c55270ed752aa26 (diff) | |
| download | linux-cc6b54aa54bf40b762cab45a9fc8aa81653146eb.tar.xz | |
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 'ipc')
0 files changed, 0 insertions, 0 deletions