From 0aa84140f0d540d2edfa039508858c32c266c368 Mon Sep 17 00:00:00 2001 From: P33M Date: Mon, 5 Aug 2013 13:17:58 +0100 Subject: [PATCH] dwc_otg: prevent leaking URBs during enqueue A dwc_otg_urb would get leaked if the HCD enqueue function failed for any reason. Free the URB at the appropriate points. --- drivers/usb/host/dwc_otg/dwc_otg_hcd_linux.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/drivers/usb/host/dwc_otg/dwc_otg_hcd_linux.c b/drivers/usb/host/dwc_otg/dwc_otg_hcd_linux.c index ae4271a6bb260a..ee8eec9c6f79c1 100644 --- a/drivers/usb/host/dwc_otg/dwc_otg_hcd_linux.c +++ b/drivers/usb/host/dwc_otg/dwc_otg_hcd_linux.c @@ -797,11 +797,19 @@ static int dwc_otg_urb_enqueue(struct usb_hcd *hcd, #if USB_URB_EP_LINKING usb_hcd_unlink_urb_from_ep(hcd, urb); #endif + DWC_FREE(dwc_otg_urb); urb->hcpriv = NULL; if (retval == -DWC_E_NO_DEVICE) retval = -ENODEV; } } +#if USB_URB_EP_LINKING + else + { + DWC_FREE(dwc_otg_urb); + urb->hcpriv = NULL; + } +#endif DWC_SPINUNLOCK_IRQRESTORE(dwc_otg_hcd->lock, irqflags); return retval; }