usb: dwc3: gadget: WARN() on bogus usb_ep_queue()
authorFelipe Balbi <balbi@ti.com>
Mon, 13 Oct 2014 20:36:16 +0000 (15:36 -0500)
committerFelipe Balbi <balbi@ti.com>
Mon, 3 Nov 2014 16:01:00 +0000 (10:01 -0600)
Some gadget/function drivers might want to do
improper request recycling by allocating a single
request from one particular endpoint and queueing
it to another completely unrelated endpoint.

One such case was found with f_loopback.c.

To prevent such cases from happening again, let's
WARN() so we get a loud enough failure and persuade
users to report errors.

Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/dwc3/gadget.c

index 12f42842da10230d6fda43a8a62c863fd6f23711..20e4ee922c47f974ae52fe14d914368355d637f8 100644 (file)
@@ -1140,8 +1140,14 @@ static int dwc3_gadget_ep_queue(struct usb_ep *ep, struct usb_request *request,
        if (!dep->endpoint.desc) {
                dev_dbg(dwc->dev, "trying to queue request %p to disabled %s\n",
                                request, ep->name);
-               spin_unlock_irqrestore(&dwc->lock, flags);
-               return -ESHUTDOWN;
+               ret = -ESHUTDOWN;
+               goto out;
+       }
+
+       if (WARN(req->dep != dep, "request %p belongs to '%s'\n",
+                               request, req->dep->name)) {
+               ret = -EINVAL;
+               goto out;
        }
 
        dev_vdbg(dwc->dev, "queing request %p to %s length %d\n",
@@ -1149,6 +1155,8 @@ static int dwc3_gadget_ep_queue(struct usb_ep *ep, struct usb_request *request,
        trace_dwc3_ep_queue(req);
 
        ret = __dwc3_gadget_ep_queue(dep, req);
+
+out:
        spin_unlock_irqrestore(&dwc->lock, flags);
 
        return ret;