Commit 8027a9ad authored by Jiasheng Jiang's avatar Jiasheng Jiang Committed by Philipp Zabel

drm/imx: imx-ldb: Check for null pointer after calling kmemdup

As the possible failure of the allocation, kmemdup() may return NULL
pointer.
Therefore, it should be better to check the return value of kmemdup()
and return error if fails.

Fixes: dc80d703 ("drm/imx-ldb: Add support to drm-bridge")
Signed-off-by: default avatarJiasheng Jiang <jiasheng@iscas.ac.cn>
Signed-off-by: default avatarPhilipp Zabel <p.zabel@pengutronix.de>
Link: https://lore.kernel.org/r/20220105074729.2363657-1-jiasheng@iscas.ac.cn
parent 31231092
...@@ -572,6 +572,8 @@ static int imx_ldb_panel_ddc(struct device *dev, ...@@ -572,6 +572,8 @@ static int imx_ldb_panel_ddc(struct device *dev,
edidp = of_get_property(child, "edid", &edid_len); edidp = of_get_property(child, "edid", &edid_len);
if (edidp) { if (edidp) {
channel->edid = kmemdup(edidp, edid_len, GFP_KERNEL); channel->edid = kmemdup(edidp, edid_len, GFP_KERNEL);
if (!channel->edid)
return -ENOMEM;
} else if (!channel->panel) { } else if (!channel->panel) {
/* fallback to display-timings node */ /* fallback to display-timings node */
ret = of_get_drm_display_mode(child, ret = of_get_drm_display_mode(child,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment