On Mon, Apr 30, 2018 at 11:24:48AM -0700, Greg Kroah-Hartman wrote:
On Mon, Apr 30, 2018 at 01:11:34PM -0500, Bin Liu wrote:
On Mon, Apr 30, 2018 at 09:42:15AM -0700, Greg Kroah-Hartman wrote:
On Mon, Apr 30, 2018 at 11:20:53AM -0500, Bin Liu wrote:
musb_start_urb() doesn't check the pass-in parameter if it is NULL. But in musb_bulk_nak_timeout() the parameter passed to musb_start_urb() is returned from first_qh(), which could be NULL.
So wrap the musb_start_urb() call here with a if condition check to avoid the potential NULL pointer dereference.
Fixes: f283862f3b5cb("usb: musb: NAK timeout scheme on bulk TX endpoint")
Nit, you forgot a ' ', this should be: f283862f3b5c ("usb: musb: NAK timeout scheme on bulk TX endpoint")
Sorry, thanks.
You also had one extra id value in there, odd. I'll edit this by
Not sure why 'git blame' gives that one extra on my computer. I will see if I will figure it out...
Why use 'git blame'? I use: git show -s --abbrev-commit --abbrev=12 --pretty=format:"%h ("%s")%n"
to generate these types of lines.
My ~/.gitconfig already has abbrev = 12
but I used 'git blame' to find the commit id which introduces the bug, then just directly copied the commit id from there... only until now know 'git blame' gives 13 chars...
I will create a shortcut for the command you gave above to avoid this 13 chars problem from now on.
Thanks, -Bin.