diff options
| author |   Jean-Baptiste Maneyrol <jmaneyrol@invensense.com> | 2014-11-20 00:46:37 +0800 |
|---|---|---|
| committer |   Jiri Kosina <jkosina@suse.cz> | 2014-11-25 15:24:54 +0100 |
| commit | 6296f4a8eb86f9abcc370fb7a1a116b8441c17fd (patch) | |
| tree | bea704c7efa5eda1c5ca33271b44e7a4d34df272 /drivers/hid/wacom_sys.c | |
| parent | HID: i2c-hid: print the correct data in dbg msg (diff) | |
| download | linux-dev-6296f4a8eb86f9abcc370fb7a1a116b8441c17fd.tar.xz linux-dev-6296f4a8eb86f9abcc370fb7a1a116b8441c17fd.zip | |
HID: i2c-hid: fix race condition reading reports
Current driver uses a common buffer for reading reports either
synchronously in i2c_hid_get_raw_report() and asynchronously in
the interrupt handler.
There is race condition if an interrupt arrives immediately after
the report is received in i2c_hid_get_raw_report(); the common
buffer is modified by the interrupt handler with the new report
and then i2c_hid_get_raw_report() proceed using wrong data.
Fix it by using a separate buffers for synchronous reports.
Signed-off-by: Jean-Baptiste Maneyrol <jmaneyrol@invensense.com>
[Antonio Borneo: cleanup, rebase to v3.17, submit mainline]
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Cc: stable@vger.kernel.org
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid/wacom_sys.c')
0 files changed, 0 insertions, 0 deletions