diff options
author | Valentin Ilie <valentin.ilie@gmail.com> | 2013-10-24 09:14:22 -0400 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2013-10-24 12:46:15 -0400 |
commit | 4b6271a64463f4fcbaf8b2e1d84704b7eb8c407c (patch) | |
tree | fed7c03ad0ff015ade5f52ad273580656788b57f /drivers/dma/edma.c | |
parent | 18ebd564e45eacd349daa31cf865183d578653d7 (diff) |
dma: edma: Fix memory leak
When it fails to allocate a slot, edesc should be free'd before return;
Signed-off-by: Valentin Ilie <valentin.ilie@gmail.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma/edma.c')
-rw-r--r-- | drivers/dma/edma.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/dma/edma.c b/drivers/dma/edma.c index 3519111c566b..134fa96de1b3 100644 --- a/drivers/dma/edma.c +++ b/drivers/dma/edma.c | |||
@@ -305,6 +305,7 @@ static struct dma_async_tx_descriptor *edma_prep_slave_sg( | |||
305 | edma_alloc_slot(EDMA_CTLR(echan->ch_num), | 305 | edma_alloc_slot(EDMA_CTLR(echan->ch_num), |
306 | EDMA_SLOT_ANY); | 306 | EDMA_SLOT_ANY); |
307 | if (echan->slot[i] < 0) { | 307 | if (echan->slot[i] < 0) { |
308 | kfree(edesc); | ||
308 | dev_err(dev, "Failed to allocate slot\n"); | 309 | dev_err(dev, "Failed to allocate slot\n"); |
309 | kfree(edesc); | 310 | kfree(edesc); |
310 | return NULL; | 311 | return NULL; |