diff options
author | Andrew Lunn <andrew@lunn.ch> | 2015-01-19 16:00:15 -0600 |
---|---|---|
committer | Andrew Lunn <andrew@lunn.ch> | 2015-01-19 16:00:15 -0600 |
commit | fe6e91e3387e4f90ed4605a1a538248a31686c16 (patch) | |
tree | d4e10d0d214ca73500fcbb3dbbd25027320fddda /drivers/hid/hid-logitech-dj.c | |
parent | c6574542149dede8093e326d2c358ba447e30f33 (diff) | |
parent | 38bdf45f4aa5cb6186d50a29e6cbbd9d486a1519 (diff) |
Merge branch 'mvebu/fixes-3' into mvebu/soc
Diffstat (limited to 'drivers/hid/hid-logitech-dj.c')
-rw-r--r-- | drivers/hid/hid-logitech-dj.c | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/drivers/hid/hid-logitech-dj.c b/drivers/hid/hid-logitech-dj.c index c917ab61aaf..5bc6d80d5be 100644 --- a/drivers/hid/hid-logitech-dj.c +++ b/drivers/hid/hid-logitech-dj.c @@ -962,10 +962,24 @@ static int logi_dj_raw_event(struct hid_device *hdev, switch (data[0]) { case REPORT_ID_DJ_SHORT: + if (size != DJREPORT_SHORT_LENGTH) { + dev_err(&hdev->dev, "DJ report of bad size (%d)", size); + return false; + } return logi_dj_dj_event(hdev, report, data, size); case REPORT_ID_HIDPP_SHORT: - /* intentional fallthrough */ + if (size != HIDPP_REPORT_SHORT_LENGTH) { + dev_err(&hdev->dev, + "Short HID++ report of bad size (%d)", size); + return false; + } + return logi_dj_hidpp_event(hdev, report, data, size); case REPORT_ID_HIDPP_LONG: + if (size != HIDPP_REPORT_LONG_LENGTH) { + dev_err(&hdev->dev, + "Long HID++ report of bad size (%d)", size); + return false; + } return logi_dj_hidpp_event(hdev, report, data, size); } |