[PATCH 4.19] dmaengine: at_hdmac: Fix memory leak

Ludovic Desroches ludovic.desroches at microchip.com
Wed Sep 23 02:39:36 EDT 2020


On Sun, Sep 20, 2020 at 10:28:38AM +0200, Pavel Machek wrote:
> Date: Sun, 20 Sep 2020 10:28:38 +0200
> From: Pavel Machek <pavel at ucw.cz>
> To: linux-kernel at vger.kernel.org, dmaengine at vger.kernel.org,
>  linux-arm-kernel at lists.infradead.org, dan.j.williams at intel.com,
>  vkoul at kernel.org, ludovic.desroches at microchip.com, stable at vger.kernel.org,
>  Greg KH <greg at kroah.com>
> Subject: [PATCH 4.19] dmaengine: at_hdmac: Fix memory leak
> 
> 
> This fixes memory leak in at_hdmac. Mainline does not have the same
> problem.
> 
> Signed-off-by: Pavel Machek (CIP) <pavel at denx.de>
Acked-by: Ludovic Desroches <ludovic.desroches at microchip.com>

Thanks.

> 
> diff --git a/drivers/dma/at_hdmac.c b/drivers/dma/at_hdmac.c
> index 86427f6ba78c..0847b2055857 100644
> --- a/drivers/dma/at_hdmac.c
> +++ b/drivers/dma/at_hdmac.c
> @@ -1714,8 +1714,10 @@ static struct dma_chan *at_dma_xlate(struct of_phandle_args *dma_spec,
>  	atslave->dma_dev = &dmac_pdev->dev;
>  
>  	chan = dma_request_channel(mask, at_dma_filter, atslave);
> -	if (!chan)
> +	if (!chan) {
> +		kfree(atslave);
>  		return NULL;
> +	}
>  
>  	atchan = to_at_dma_chan(chan);
>  	atchan->per_if = dma_spec->args[0] & 0xff;
> 
> -- 
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html






More information about the linux-arm-kernel mailing list