diff mbox

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

Message ID 1386536638-4450-1-git-send-email-ps.report@gmx.net (mailing list archive)
State New, archived
Headers show

Commit Message

Peter Seiderer Dec. 8, 2013, 9:03 p.m. UTC 
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(+)

Comments

Sascha Hauer Dec. 9, 2013, 8:35 a.m. UTC | #1 
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
>
diff mbox

Patch

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;