diff options
| author | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2014-08-22 16:16:05 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-10-05 14:52:12 -0700 |
| commit | c84e10787d1dc2b5ccdaf7d62f5a00cae90a0adb (patch) | |
| tree | a38cc772468b18b47dad49cfc2beecc0f6fffca2 /drivers/hid/hid-gyration.c | |
| parent | 6e4106ec619b16593d66ad6384f6f983d423ee0b (diff) | |
HID: logitech-dj: prevent false errors to be shown
commit 5abfe85c1d4694d5d4bbd13ecc166262b937adf0 upstream.
Commit "HID: logitech: perform bounds checking on device_id early
enough" unfortunately leaks some errors to dmesg which are not real
ones:
- if the report is not a DJ one, then there is not point in checking
the device_id
- the receiver (index 0) can also receive some notifications which
can be safely ignored given the current implementation
Move out the test regarding the report_id and also discards
printing errors when the receiver got notified.
Fixes: ad3e14d7c5268c2e24477c6ef54bbdf88add5d36
Reported-and-tested-by: Markus Trippelsdorf <markus@trippelsdorf.de>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/hid/hid-gyration.c')
0 files changed, 0 insertions, 0 deletions