[U-Boot] [PATCH] USB-CDC: wrong ep status used

Sergei Shtylyov sshtylyov at mvista.com
Thu Aug 12 19:05:53 CEST 2010


Hello.

Stefano Babic wrote:

>> On 08/12/2010 01:41 AM, Stefano Babic wrote:
>>>  #if defined(DEV_CONFIG_CDC)
>>>  	if (dev->status_ep) {
>>> -		dev->stat_req = usb_ep_alloc_request(gadget->ep0, GFP_KERNEL);
>>> -		dev->stat_req->buf = status_req;
>>> +		dev->stat_req = usb_ep_alloc_request(dev->status_ep, GFP_KERNEL);
>>>  		if (!dev->stat_req) {
>>>  			dev->stat_req->buf=NULL;
>> We get oops here!

> Agree, and the issue is not related to this patch, I missed to correct
> it, thanks. If no one complains, I will send a single patch to fix both
> problems (wrong ep status + null pointer access).

    Looks like Vitaly has done the same already:

http://lists.denx.de/pipermail/u-boot/2010-August/075419.html

> Best regards,
> Stefano

WBR, Sergei


More information about the U-Boot mailing list