[1/2] usb: musb: host: fix potential NULL pointer dereference

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

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.

thanks,

greg k-h
---
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html