linux-kernel - Re: drm/v3d: remove duplicated kfree in v3d_submit_cl

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <55da3be6-ea90-e554-775a-de610b8fbf7b@web.de>
Date:   Sun, 29 Dec 2019 13:14:03 +0100
From:   Markus Elfring <Markus.Elfring@....de>
To:     yu kuai <yukuai3@...wei.com>, Daniel Vetter <daniel@...ll.ch>,
        Navid Emamdoost <navid.emamdoost@...il.com>,
        Eric Anholt <eric@...olt.net>, dri-devel@...ts.freedesktop.org
Cc:     linux-kernel@...r.kernel.org, David Airlie <airlied@...ux.ie>,
        Yi Zhang <yi.zhang@...wei.com>,
        zhengbin <zhengbin13@...wei.com>, Kangjie Lu <kjlu@....edu>,
        Navid Emamdoost <emamd001@....edu>
Subject: Re: drm/v3d: remove duplicated kfree in v3d_submit_cl_ioctl

>> 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@...il.com>
> 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@gmail.com/
https://lore.kernel.org/patchwork/comment/1342088/
https://lore.kernel.org/patchwork/patch/1142603/


Thus the fix chain will be extended.

Regards,
Markus