From: Felipe Balbi <balbi@ti.com>
Date: Thu, 8 Nov 2012 13:26:41 +0000 (+0200)
Subject: usb: dwc3: core: don't kfree() devm_kzalloc()'ed memory
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=e32672f0bc7ec8421aa8e580e9f2216d3bec2505;p=deliverable%2Flinux.git

usb: dwc3: core: don't kfree() devm_kzalloc()'ed memory

commit 380f0d2 (usb: dwc3: core: switch event
buffer allocation to devm_kzalloc()) was incomplete
leaving a trailing kfree(evt) in an error exit
path.

Fix this problem by removing the trailing kfree(evt).

Cc: Julia Lawall <julia.lawall@lip6.fr>
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
---

diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c
index d8d327a5e53b..2bd007d16461 100644
--- a/drivers/usb/dwc3/core.c
+++ b/drivers/usb/dwc3/core.c
@@ -153,10 +153,8 @@ dwc3_alloc_one_event_buffer(struct dwc3 *dwc, unsigned length)
 	evt->length	= length;
 	evt->buf	= dma_alloc_coherent(dwc->dev, length,
 			&evt->dma, GFP_KERNEL);
-	if (!evt->buf) {
-		kfree(evt);
+	if (!evt->buf)
 		return ERR_PTR(-ENOMEM);
-	}
 
 	return evt;
 }