diff options
| author |   Jiasheng Jiang <jiasheng@iscas.ac.cn> | 2022-01-05 15:47:29 +0800 |
|---|---|---|
| committer |   Philipp Zabel <p.zabel@pengutronix.de> | 2022-04-04 09:37:41 +0200 |
| commit | 8027a9ad9b3568c5eb49c968ad6c97f279d76730 (patch) | |
| tree | a31477129dea76d890550fb5d2f2edde947701cc /drivers/gpu/drm/imx/imx-ldb.c | |
| parent | Linux 5.18-rc1 (diff) | |
| download | linux-dev-8027a9ad9b3568c5eb49c968ad6c97f279d76730.tar.xz linux-dev-8027a9ad9b3568c5eb49c968ad6c97f279d76730.zip | |
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: dc80d7038883 ("drm/imx-ldb: Add support to drm-bridge")
Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn>
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Link: https://lore.kernel.org/r/20220105074729.2363657-1-jiasheng@iscas.ac.cn
Diffstat (limited to 'drivers/gpu/drm/imx/imx-ldb.c')
| -rw-r--r-- | drivers/gpu/drm/imx/imx-ldb.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/drm/imx/imx-ldb.c b/drivers/gpu/drm/imx/imx-ldb.c index e5078d03020d..fb0e951248f6 100644 --- a/drivers/gpu/drm/imx/imx-ldb.c +++ b/drivers/gpu/drm/imx/imx-ldb.c @@ -572,6 +572,8 @@ static int imx_ldb_panel_ddc(struct device *dev, edidp = of_get_property(child, "edid", &edid_len); if (edidp) { channel->edid = kmemdup(edidp, edid_len, GFP_KERNEL); + if (!channel->edid) + return -ENOMEM; } else if (!channel->panel) { /* fallback to display-timings node */ ret = of_get_drm_display_mode(child, |