[PATCH v2] imx-drm: fix channel->edid memleak

[PATCH v2] imx-drm: fix channel->edid memleak

[Peter Seiderer]

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
Changes v1 -> v2:
  - add missing Signed-off-by line

 drivers/staging/imx-drm/imx-ldb.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/staging/imx-drm/imx-ldb.c b/drivers/staging/imx-drm/imx-ldb.c
index 7e59329..448f3ce 100644
--- a/drivers/staging/imx-drm/imx-ldb.c
+++ b/drivers/staging/imx-drm/imx-ldb.c
@@ -599,6 +599,8 @@ static int imx_ldb_remove(struct platform_device *pdev)
 
 		imx_drm_remove_connector(channel->imx_drm_connector);
 		imx_drm_remove_encoder(channel->imx_drm_encoder);
+
+		kfree(channel->edid);
 	}
 
 	return 0;
-- 
1.8.1.4

[PATCH v2] imx-drm: fix imxpd-->edid memleak

[Peter Seiderer]

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
Changes v1 -> v2:
  - add missing Signed-off-by line

 drivers/staging/imx-drm/parallel-display.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/staging/imx-drm/parallel-display.c b/drivers/staging/imx-drm/parallel-display.c
index 24aa9be..705a914 100644
--- a/drivers/staging/imx-drm/parallel-display.c
+++ b/drivers/staging/imx-drm/parallel-display.c
@@ -248,6 +248,8 @@ static int imx_pd_remove(struct platform_device *pdev)
 	imx_drm_remove_connector(imxpd->imx_drm_connector);
 	imx_drm_remove_encoder(imxpd->imx_drm_encoder);
 
+	kfree(imxpd->edid);
+
 	return 0;
 }
 
-- 
1.8.1.4

Re: [PATCH v2] imx-drm: fix channel->edid memleak

[Sascha Hauer]

On Sun, Dec 08, 2013 at 10:03:57PM +0100, Peter Seiderer wrote:
> 
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
> Changes v1 -> v2:
>   - add missing Signed-off-by line
> 
>  drivers/staging/imx-drm/imx-ldb.c | 2 ++
>  1 file changed, 2 insertions(+)

Acked-by: Sascha Hauer <s.hauer@pengutronix.de>

Sascha

> 
> diff --git a/drivers/staging/imx-drm/imx-ldb.c b/drivers/staging/imx-drm/imx-ldb.c
> index 7e59329..448f3ce 100644
> --- a/drivers/staging/imx-drm/imx-ldb.c
> +++ b/drivers/staging/imx-drm/imx-ldb.c
> @@ -599,6 +599,8 @@ static int imx_ldb_remove(struct platform_device *pdev)
>  
>  		imx_drm_remove_connector(channel->imx_drm_connector);
>  		imx_drm_remove_encoder(channel->imx_drm_encoder);
> +
> +		kfree(channel->edid);
>  	}
>  
>  	return 0;
> -- 
> 1.8.1.4
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |