Krzysztof Opasiak <kopasia...@gmail.com> writes:
> Since commit ba1582f22231 ("usb: gadget: f_hid: use alloc_ep_req()")
> we cannot allocate any requests in bind() as we check if we should
> align request buffer based on endpoint descriptor which is assigned
> in set_alt().
>
> Allocating request in bind() function causes a NULL pointer
> dereference.
>
> This commit moves allocation of IN request from bind() to set_alt()
> to prevent this issue.
>
> Fixes: ba1582f22231 ("usb: gadget: f_hid: use alloc_ep_req()")
> Cc: sta...@vger.kernel.org
> Signed-off-by: Krzysztof Opasiak <k.opas...@samsung.com>please rebase on testing/next: checking file drivers/usb/gadget/function/f_hid.c Hunk #1 succeeded at 338 (offset 54 lines). Hunk #2 succeeded at 348 (offset 54 lines). Hunk #3 succeeded at 363 (offset 54 lines). Hunk #4 succeeded at 376 (offset 54 lines). Hunk #5 succeeded at 611 (offset 54 lines). Hunk #6 succeeded at 648 (offset 54 lines). Hunk #7 FAILED at 611. Hunk #8 succeeded at 686 (offset 54 lines). Hunk #9 succeeded at 765 (offset 54 lines). Hunk #10 succeeded at 802 (offset 65 lines). Hunk #11 succeeded at 1068 (offset 65 lines). 1 out of 11 hunks FAILED previous 3 patches applied fine, though. -- balbi
signature.asc
Description: PGP signature
