linux-kernel - Re: firmware: tegra: bpmp: Fix freeing uninitialized pointers

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>] [day] [month] [year] [list]

Message-ID: <5e798fd6-5662-48a3-8e3a-758f250828c8@web.de>
Date: Sat, 5 Oct 2024 18:35:09 +0200
From: Markus Elfring <Markus.Elfring@....de>
To: Zichen Xie <zichenxie0106@...il.com>, linux-tegra@...r.kernel.org
Cc: LKML <linux-kernel@...r.kernel.org>, kernel-janitors@...r.kernel.org,
 Chenyuan Yang <chenyuan0y@...il.com>, Jonathan Hunter
 <jonathanh@...dia.com>, Krzysztof Kozlowski
 <krzysztof.kozlowski@...aro.org>, Sumit Gupta <sumitg@...dia.com>,
 Thierry Reding <thierry.reding@...il.com>, Zijie Zhao <zzjas98@...il.com>
Subject: Re: firmware: tegra: bpmp: Fix freeing uninitialized pointers

> Thank you for the explanation and sorry for not realizing the correct
> deallocation function.
>
> I found in another email thread that the NULL initialization following
> the __free() was mainly for readability but not safety. Sorry for the
> confusion.
>
> We are working on a static analysis tool and wanted to report
> potential bugs. We directly sent a patch trying to reduce developer’s
> burden but didn’t not realize the correct use of Reported-by tag. We
> will be more careful in the future.

Please take another look at another corresponding public information source.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/researcher-guidelines.rst?h=v6.12-rc1#n5

Regards,
Markus