On Wed, Nov 13, 2013 at 10:44:46AM +0300, Dan Carpenter wrote:
> We accidentally applied two correct but duplicative fixes for a memory
> leak here:
> 4b6271a64463 ('dma: edma: Fix memory leak')
> 2f6d8fad0a16 ('dma: edma.c: remove edma_desc leakage')
Thanks Dan,I already send the same fix today morning. This was reported by Wing-Keung sometime back -- ~Vinod > > Signed-off-by: Dan Carpenter <[email protected]> > > diff --git a/drivers/dma/edma.c b/drivers/dma/edma.c > index ea4abaa..9c8103d 100644 > --- a/drivers/dma/edma.c > +++ b/drivers/dma/edma.c > @@ -420,7 +420,6 @@ static struct dma_async_tx_descriptor *edma_prep_slave_sg( > edma_alloc_slot(EDMA_CTLR(echan->ch_num), > EDMA_SLOT_ANY); > if (echan->slot[i] < 0) { > - kfree(edesc); > dev_err(dev, "Failed to allocate slot\n"); > kfree(edesc); > return NULL; > -- > To unsubscribe from this list: send the line "unsubscribe dmaengine" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html -- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
