diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-09-26 12:31:06 +0300 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2015-09-30 19:20:21 +0300 |
commit | 7022cfdfe07dcfccc294f04de5f75e0a37b16174 (patch) | |
tree | 63326523e3d837c5adf52e293c0a5db8fd709d60 /drivers/usb/gadget/udc/bdc | |
parent | 9ffecb10283508260936b96022d4ee43a7798b4c (diff) | |
download | linux-7022cfdfe07dcfccc294f04de5f75e0a37b16174.tar.xz |
usb: gadget: bdc: fix memory leak
If dma_pool_alloc() fails we are jumping to fail and releasing all the
bd_tables which have been added to the chain but we missed freeing this
bd_table which was just allocated and still not added to the chain of
bd_table.
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb/gadget/udc/bdc')
-rw-r--r-- | drivers/usb/gadget/udc/bdc/bdc_ep.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/usb/gadget/udc/bdc/bdc_ep.c b/drivers/usb/gadget/udc/bdc/bdc_ep.c index d1b81539d632..d6199507f861 100644 --- a/drivers/usb/gadget/udc/bdc/bdc_ep.c +++ b/drivers/usb/gadget/udc/bdc/bdc_ep.c @@ -159,8 +159,10 @@ static int ep_bd_list_alloc(struct bdc_ep *ep) bd_table->start_bd = dma_pool_alloc(bdc->bd_table_pool, GFP_ATOMIC, &dma); - if (!bd_table->start_bd) + if (!bd_table->start_bd) { + kfree(bd_table); goto fail; + } bd_table->dma = dma; |