On Thu, Mar 22, 2018 at 02:53:11PM -0500, Bin Liu wrote:
From: Krzysztof Opasiak kopasiak90@gmail.com
commit 749494b6bdbbaf0899aa1c62a1ad74cd747bce47 upstream.
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()") Tested-by: David Lechner david@lechnology.com Signed-off-by: Krzysztof Opasiak k.opasiak@samsung.com Signed-off-by: Felipe Balbi felipe.balbi@linux.intel.com Signed-off-by: Greg Kroah-Hartman gregkh@linuxfoundation.org
This backport is for v4.9 only.
Thanks, now queued up.
greg k-h