Re: drm/imx: Checking a kmemdup() call in imx_pd_bind()

From: Navid Emamdoost
Date: Mon Oct 07 2019 - 00:26:38 EST


Hi Markus,

I agree with you, kmemdup may fail so a null check seems necessary there.

On Sun, Oct 6, 2019 at 4:33 AM Markus Elfring <Markus.Elfring@xxxxxx> wrote:
>
> I have taken another look also at the implementation of the function âimx_pd_bindâ.
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/gpu/drm/imx/parallel-display.c?id=43b815c6a8e7dbccb5b8bd9c4b099c24bc22d135#n197
> https://elixir.bootlin.com/linux/v5.4-rc1/source/drivers/gpu/drm/imx/parallel-display.c#L197
>
> Now I find an unchecked call of the function âkmemdupâ suspicious.
> Will this detail trigger further software development considerations?
>
> Regards,
> Markus



--
Navid.