feedc0de/esp-idf
1
0
Fork 0
forked from espressif/esp-idf
Code Pull Requests Activity

fix(esp_hid): Loose the check of input report length to be more compatible

Browse Source 
Closes https://github.com/espressif/esp-idf/issues/14259
This commit is contained in:
liqigan
2024-08-07 10:10:26 +08:00
parent aeffb97301
commit f0add1b1c1
1 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
components/esp_hid/src/esp_hidh.c
View File

@ -1,5 +1,5 @@
/* /*
* SPDX-FileCopyrightText: 2017-2023 Espressif Systems (Shanghai) CO LTD * SPDX-FileCopyrightText: 2017-2024 Espressif Systems (Shanghai) CO LTD
* *
* SPDX-License-Identifier: Apache-2.0 * SPDX-License-Identifier: Apache-2.0
*/ */
@ -518,7 +518,7 @@ esp_err_t esp_hidh_dev_report_maps_get(esp_hidh_dev_t *dev, size_t *num_maps, es
* */ * */
/** /**
* `lock_devices()` only protect the devices list, this mutex protect the single deivce instance. * `lock_devices()` only protect the devices list, this mutex protect the single device instance.
*/ */
inline void esp_hidh_dev_lock(esp_hidh_dev_t *dev) inline void esp_hidh_dev_lock(esp_hidh_dev_t *dev)
{ {
@ -637,8 +637,13 @@ esp_hidh_dev_report_t *esp_hidh_dev_get_input_report_by_proto_and_data(esp_hidh_
} }
r = dev->reports; r = dev->reports;
while (r) { while (r) {
if (r->value_len == len - 1 && r->report_id == *data && (r->report_type & ESP_HID_REPORT_TYPE_INPUT) && /**
r->protocol_mode == protocol_mode) { * @note For some HID device, the input report length may exceed the length which is declared in HID
* descriptor, like Logitech K380 keyboard. So loose the check condition from `r->value_len == len - 1` to
* `r->value_len <= len - 1`.
*/
if (r->value_len <= len - 1 && r->report_id == *data && (r->report_type & ESP_HID_REPORT_TYPE_INPUT) &&
r->protocol_mode == protocol_mode) {
*has_report_id = true; *has_report_id = true;
break; break;
} }