UPSTREAM: HID: playstation: fix array size comparison (off-by-one)
The comparison of value with the array size ps_gamepad_hat_mapping appears to be off-by-one. Fix this by using >= rather than > for the size comparison. Addresses-Coverity: ("Out-of-bounds read") Fixes: bc2e15a9a022 ("HID: playstation: initial DualSense USB support.") Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com> Bug: 167947264 Change-Id: I24f4c7f8d48ab8ff1d117d33c7914b8d2e4292f8 (cherry picked from commit 50ab1ffd7c41c5c7759b62fb42d3006b751bb12b) Signed-off-by: Farid Chahla <farid.chahla@sony.com>
This commit is contained in:
parent
8e0061c2c3
commit
6f239ba207
@ -845,7 +845,7 @@ static int dualsense_parse_report(struct ps_device *ps_dev, struct hid_report *r
|
||||
input_report_abs(ds->gamepad, ABS_RZ, ds_report->rz);
|
||||
|
||||
value = ds_report->buttons[0] & DS_BUTTONS0_HAT_SWITCH;
|
||||
if (value > ARRAY_SIZE(ps_gamepad_hat_mapping))
|
||||
if (value >= ARRAY_SIZE(ps_gamepad_hat_mapping))
|
||||
value = 8; /* center */
|
||||
input_report_abs(ds->gamepad, ABS_HAT0X, ps_gamepad_hat_mapping[value].x);
|
||||
input_report_abs(ds->gamepad, ABS_HAT0Y, ps_gamepad_hat_mapping[value].y);
|
||||
|
Loading…
Reference in New Issue
Block a user