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

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