Re: [PATCH v2 2/3] EDAC/versalnet: Fix device name memory leak
From: Borislav Petkov
Date: Tue May 05 2026 - 09:24:09 EST
On Wed, Apr 01, 2026 at 04:18:56AM -0700, Prasanna Kumar T S M wrote:
> The device name allocated via kzalloc() in init_one_mc() is assigned to
> dev->init_name but never freed on the normal removal path.
> device_register() copies init_name and then sets dev->init_name to NULL,
> so the name pointer becomes unreachable from the device. Thus leaking
> memory.
>
> Use a stack-local char array instead of using kzalloc() for name.
>
> Fixes: d5fe2fec6c40 ("EDAC: Add a driver for the AMD Versal NET DDR controller")
> Cc: stable@xxxxxxxxxxxxxxx
> Signed-off-by: Prasanna Kumar T S M <ptsm@xxxxxxxxxxxxxxxxxxx>
> ---
> drivers/edac/versalnet_edac.c | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
Applied, thanks.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette