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 /fs/openpromfs | |
parent | 0a9cd0a80ac559357c6a90d26c55270ed752aa26 (diff) | |
download | op-kernel-dev-cc6b54aa54bf40b762cab45a9fc8aa81653146eb.zip op-kernel-dev-cc6b54aa54bf40b762cab45a9fc8aa81653146eb.tar.gz |
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 'fs/openpromfs')
0 files changed, 0 insertions, 0 deletions