[PATCH] video/edid: Fix edid_to_display_timings error path

Philipp Zabel p.zabel at pengutronix.de
Thu Sep 1 02:28:39 PDT 2016

edid_to_display_timings is called in drivers/video/imx-ipu-v3/imx-hdmi.c,
drivers/video/simple-panel.c, and drivers/video/edid.c with the struct
display_timings *timings parameter pointing to an embedded struct
display_timings inside a struct fb_info. The timings pointer is not to
be freed in the error path.

Signed-off-by: Philipp Zabel <p.zabel at pengutronix.de>
 drivers/video/edid.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/video/edid.c b/drivers/video/edid.c
index 92b59a5..2585264 100644
--- a/drivers/video/edid.c
+++ b/drivers/video/edid.c
@@ -792,7 +792,6 @@ int edid_to_display_timings(struct display_timings *timings, unsigned char *edid
 	return 0;
-	free(timings);
 	return ret;

More information about the barebox mailing list