Re: drm/v3d: remove duplicated kfree in v3d_submit_cl_ioctl

[Markus Elfring]

>> Please add the tag âFixesâ to your change description.
>
> I got the results from "git blame":
> git blame -L 570,575 drivers/gpu/drm/v3d/v3d_gem.c
â
> 0d352a3a8a1f2 (Iago Toral Quiroga 2019-09-16 09:11:25 +0200 571) ÂÂÂÂÂÂÂÂÂÂÂÂÂÂ kfree(bin);
> a783a09ee76d6 (Eric AnholtÂÂÂÂÂÂÂ 2019-04-16 15:58:53 -0700 572) ÂÂÂÂÂÂÂÂÂÂÂÂÂÂ v3d_job_put(&render->base);
> 29cd13cfd7624 (Navid EmamdoostÂÂÂ 2019-10-21 13:52:49 -0500 573) ÂÂÂÂÂÂÂÂÂÂÂÂÂÂ kfree(bin);
> a783a09ee76d6 (Eric AnholtÂÂÂÂÂÂÂ 2019-04-16 15:58:53 -0700 574) ÂÂÂÂÂÂÂÂÂÂÂÂÂÂ return ret;
â
>
â
> commit 29cd13cfd7624726d9e6becbae9aa419ef35af7f
> Author: Navid Emamdoost <navid.emamdoost@xxxxxxxxx>
> Date:ÂÂ Mon Oct 21 13:52:49 2019 -0500
>
> ÂÂÂ drm/v3d: Fix memory leak in v3d_submit_cl_ioctl
>
> ÂÂÂ In the impelementation of v3d_submit_cl_ioctl() there are two memory
â

It seems that this patch got insufficient code review attention before.
https://lore.kernel.org/dri-devel/20191021185250.26130-1-navid.emamdoost@xxxxxxxxx/
https://lore.kernel.org/patchwork/comment/1342088/
https://lore.kernel.org/patchwork/patch/1142603/


Thus the fix chain will be extended.

Regards,
Markus