Bug in USB HID, keyboard code

[Grzegorz Kania]

Posted on July 13, 2017 at 11:18

I'd like to report a bug. 

In the file usbh_hid_keybd.c, ver. V3.2.2, date: 07-July-2015

line 333:

Now is:

if(HID_Handle->length > (sizeof(keybd_report_data)/sizeof(uint32_t))){

   HID_Handle->length = (sizeof(keybd_report_data)/sizeof(uint32_t)); }

should be replaced with:

if(HID_Handle->length > sizeof(keybd_report_data)){

   HID_Handle->length = sizeof(keybd_report_data); }

If I did not make a change, the keyboard reported an error by sending a stall event.

Can anyone confirm this?

#host-usb-hid-keyboard

[Amel NASRI]

Posted on July 17, 2017 at 16:12

Hi

Kania.Grzegorz.002

‌,

Could you please precise the FW package you are using?

-Amel

[Grzegorz Kania]

Posted on July 18, 2017 at 09:18

Hi Amel

I am using the STM32Cube_FW_F7_V1.3.0 package, but I found this file also in others, eg: STM32Cube_FW_F1_V1.4.0, STM32Cube_FW_F2_V1.6.0

Grzegorz

[Igor Pokorny]

Posted on December 22, 2017 at 00:09

Well, I do have the same experience. A small keyboard I was planning to use (Gembird Mini Usb keyboard) stalled after ennumeration until I didn't change lines according KIania's recomendation. Thanks.  Never the less the mentioned keyboard works in Linux without any problem as well as any other keyboard I tried with ST USB library.  Strange. 

I am using the last version 

STM32Cube_FW_F7_V1.8.0.

 

Igor