Re: firmware: tegra: bpmp: Fix freeing uninitialized pointers

From: Markus Elfring
Date: Sat Oct 05 2024 - 12:35:38 EST


> 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