search

STMicroelectronics / STM32CubeG0

STM32Cube MCU Full Package for the STM32G0 series - (HAL + LL Drivers, CMSIS Core, CMSIS Device, MW libraries plus a set of Projects running on all boards provided by ST (Nucleo, Evaluation and Discovery Kits))
Other
164 stars 75 forks source link

Update usbh_ctlreq.c #29

Closed DinkDonk closed 2 years ago

DinkDonk commented 2 years ago

Removed unneeded length checks that broke class-specific interface descriptors. There should be no need to force length on configuration descriptors since they all declare their own sizes with bLength.

ALABSTM commented 2 years ago

Hi @DinkDonk,

Thank you for this pull-request. We will get back to you with a feedback as soon as possible. Please excuse the delay it may take us sometimes to reply. Thank you for your comprehension.

With regards,

ALABSTM commented 2 years ago

Hi @DinkDonk,

Thank you for this proposal. However, according to our development teams, those checks are necessary to protect against potential attacks in case the inserted device does not respect USB standards. Hence, we cannot remove them.

Please allow me to close this pull-request. Thank you for you comprehension and thank you again for the proposal.

With regards,