arm: xlnx: fix minor memory leaks

[PATCH for-9.2 0/6] arm: xlnx: fix minor memory leaks

Peter Maydell posted 6 patches 3 months ago

Download series mbox

include/hw/misc/xlnx-versal-trng.h   |  1 +
include/hw/nvram/xlnx-bbram.h        |  1 +
include/hw/nvram/xlnx-versal-efuse.h |  1 +
include/hw/nvram/xlnx-zynqmp-efuse.h |  1 +
hw/misc/xlnx-versal-cfu.c            |  8 ++++++++
hw/misc/xlnx-versal-trng.c           | 12 ++++++------
hw/nvram/xlnx-bbram.c                | 13 ++++++++++---
hw/nvram/xlnx-versal-efuse-ctrl.c    |  6 +++---
hw/nvram/xlnx-zynqmp-efuse.c         | 13 ++++++++++---
9 files changed, 41 insertions(+), 15 deletions(-)

Expand all Fold all

[PATCH for-9.2 0/6] arm: xlnx: fix minor memory leaks

Posted by Peter Maydell 3 months ago

This patchset fixes a collection of minor memory leaks in
various xlnx devices, all detected by clang LeakSanitizer
when running 'make check'. Since these are longstanding
and not very important leaks, this is 9.2 material.

thanks
-- PMM

Peter Maydell (6):
  hw/misc/xlnx-versal-cfu: destroy fifo in finalize
  hw/misc/xlnx-versal-trng: Free s->prng in finalize, not unrealize
  hw/nvram/xlnx-bbram: Call register_finalize_block
  hw/nvram/xlnx-zynqmp-efuse: Call register_finalize_block
  hw/misc/xlnx-versal-trng: Call register_finalize_block
  hm/nvram/xlnx-versal-efuse-ctrl: Call register_finalize_block

 include/hw/misc/xlnx-versal-trng.h   |  1 +
 include/hw/nvram/xlnx-bbram.h        |  1 +
 include/hw/nvram/xlnx-versal-efuse.h |  1 +
 include/hw/nvram/xlnx-zynqmp-efuse.h |  1 +
 hw/misc/xlnx-versal-cfu.c            |  8 ++++++++
 hw/misc/xlnx-versal-trng.c           | 12 ++++++------
 hw/nvram/xlnx-bbram.c                | 13 ++++++++++---
 hw/nvram/xlnx-versal-efuse-ctrl.c    |  6 +++---
 hw/nvram/xlnx-zynqmp-efuse.c         | 13 ++++++++++---
 9 files changed, 41 insertions(+), 15 deletions(-)

-- 
2.34.1

Re: [PATCH for-9.2 0/6] arm: xlnx: fix minor memory leaks

Posted by Edgar E. Iglesias 3 months ago

On Thu, Aug 22, 2024 at 6:21 PM Peter Maydell <peter.maydell@linaro.org>
wrote:

> This patchset fixes a collection of minor memory leaks in
> various xlnx devices, all detected by clang LeakSanitizer
> when running 'make check'. Since these are longstanding
> and not very important leaks, this is 9.2 material.
>
>
All of it looks good to me:
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@amd.com>



> thanks
> -- PMM
>
> Peter Maydell (6):
>   hw/misc/xlnx-versal-cfu: destroy fifo in finalize
>   hw/misc/xlnx-versal-trng: Free s->prng in finalize, not unrealize
>   hw/nvram/xlnx-bbram: Call register_finalize_block
>   hw/nvram/xlnx-zynqmp-efuse: Call register_finalize_block
>   hw/misc/xlnx-versal-trng: Call register_finalize_block
>   hm/nvram/xlnx-versal-efuse-ctrl: Call register_finalize_block
>
>  include/hw/misc/xlnx-versal-trng.h   |  1 +
>  include/hw/nvram/xlnx-bbram.h        |  1 +
>  include/hw/nvram/xlnx-versal-efuse.h |  1 +
>  include/hw/nvram/xlnx-zynqmp-efuse.h |  1 +
>  hw/misc/xlnx-versal-cfu.c            |  8 ++++++++
>  hw/misc/xlnx-versal-trng.c           | 12 ++++++------
>  hw/nvram/xlnx-bbram.c                | 13 ++++++++++---
>  hw/nvram/xlnx-versal-efuse-ctrl.c    |  6 +++---
>  hw/nvram/xlnx-zynqmp-efuse.c         | 13 ++++++++++---
>  9 files changed, 41 insertions(+), 15 deletions(-)
>
> --
> 2.34.1
>
>