diff options
| author | Grigor Tovmasyan <[email protected]> | 2018-05-24 18:22:30 +0400 |
|---|---|---|
| committer | Felipe Balbi <[email protected]> | 2018-06-19 12:48:14 +0300 |
| commit | 9bb073a053f0464ea74a4d4c331fdb7da58568d6 (patch) | |
| tree | 9327045eefd467b1a6e7f95e30b3eca167fef996 | |
| parent | 980900d6318066b9f8314bfb87329a20fd0d1ca4 (diff) | |
usb: gadget: dwc2: fix memory leak in gadget_init()
Freed allocated request for ep0 to prevent memory leak in case when
dwc2_driver_probe() failed.
Cc: Stefan Wahren <[email protected]>
Cc: Marek Szyprowski <[email protected]>
Tested-by: Stefan Wahren <[email protected]>
Tested-by: Marek Szyprowski <[email protected]>
Acked-by: Minas Harutyunyan <[email protected]>
Signed-off-by: Grigor Tovmasyan <[email protected]>
Signed-off-by: Felipe Balbi <[email protected]>
| -rw-r--r-- | drivers/usb/dwc2/gadget.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb/dwc2/gadget.c index fa3b6f361074..76e4f2e7d947 100644 --- a/drivers/usb/dwc2/gadget.c +++ b/drivers/usb/dwc2/gadget.c @@ -4735,9 +4735,11 @@ int dwc2_gadget_init(struct dwc2_hsotg *hsotg) } ret = usb_add_gadget_udc(dev, &hsotg->gadget); - if (ret) + if (ret) { + dwc2_hsotg_ep_free_request(&hsotg->eps_out[0]->ep, + hsotg->ctrl_req); return ret; - + } dwc2_hsotg_dump(hsotg); return 0; @@ -4751,6 +4753,7 @@ int dwc2_gadget_init(struct dwc2_hsotg *hsotg) int dwc2_hsotg_remove(struct dwc2_hsotg *hsotg) { usb_del_gadget_udc(&hsotg->gadget); + dwc2_hsotg_ep_free_request(&hsotg->eps_out[0]->ep, hsotg->ctrl_req); return 0; } |