xiaocq2001
commented
2 years ago Thanks for the feedback.
The issue should be in _ux_host_class_hid_client_search, where linked client should not be used for new hid client. Please try following modification:
ux_host_class_hid_client_search.c, L110:
/* Check if this HID client is registered and not locally used. */
if ((hid_client -> ux_host_class_hid_client_status == UX_USED) &&
(hid_client -> ux_host_class_hid_client_local_instance != UX_NULL))
HelloByeAll
USBX HID 设备问题现象：
以鼠标举例, 假设第一个插入的鼠标为1号鼠标， 第二个插入的为2号鼠标，2个鼠标都插入后，将2号鼠标拔出，此时1号鼠标也将不能使用。
更严重的影响：发生内存踩踏
2号鼠标被拔出后，它所使用的内存将被释放，此时1号鼠标动作虽然使用api获取的属性值(x/y坐标、按键值、滚轮值)没有改变，当实际原因是1号鼠标的属性值被更新到2号鼠标原先属性值所在的内存中去了。1号鼠标仍在操作原本已经被释放的2号鼠标的内存。
LOG分析：
鼠标插入:
从LOG上可以看出插入一号鼠标后 一号鼠标的
mouse_instance = 0x2116D830接着插入2号鼠标mouse_instance = 0x21174140从下面这2条LOG可以看出 他们所属的hid客户端是同一个，也就是调用
ux_host_class_hid_client_register(_ux_system_host_class_hid_client_mouse_name, ux_host_class_hid_mouse_entry)注册鼠标客户端时malloc的一片内存。鼠标枚举LOG _ux_host_class_hid_mouse_activate
源码:
从这个.c的源码中也能看出，这里
hid_client -> ux_host_class_hid_client_local_instance = (VOID *) mouse_instance;的操作会覆盖原本的这个值, 即使这个鼠标枚举失败了hid_client -> ux_host_class_hid_client_local_instance也没有恢复操作，1号原本可以使用的对象就直接丢失了。(这里就不分析这个情况了，看代码就能理解了)LOG：
2号鼠标拔出:
从LOG中可以很清楚的看到2号鼠标已经被协议栈移除
free hid_client = 0x21163370 instance 0x21174140并且已经释放了0x21174140这个地址的内存。源码:
LOG:
2号鼠标拔出后移动1号鼠标:
从下面日志可以看到
mouse_instance = 0x21174140这个地址正是已经被释放了的2号鼠标所拥有的0x21174140而不是1号本身的0x2116D830源码中野可以看到，他下面直接对这个地址里的属性值做了修改，此时就发生了内存踩踏源码：
LOG:
hid客户端注册 ux_host_class_hid_client_register
完整日志：