On Mon, Sep 1, 2025 at 3:54 PM Alex Bennée <alex.ben...@linaro.org> wrote:
>
> This is potentially more of a bike-shed case as node_path will persist
> until the end of the function.
>
> Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
> ---
Reviewed-by: Manos Pitsidianakis <manos.pitsidiana...@linaro.org>
> hw/arm/boot.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/hw/arm/boot.c b/hw/arm/boot.c
> index 749f2d08341..f9d0bc7011e 100644
> --- a/hw/arm/boot.c
> +++ b/hw/arm/boot.c
> @@ -520,11 +520,11 @@ int arm_load_dtb(hwaddr addr, const struct
> arm_boot_info *binfo,
> ARMCPU *cpu)
> {
> g_autofree void *fdt = NULL;
> + g_auto(GStrv) node_path = NULL;
> int size, rc, n = 0;
> uint32_t acells, scells;
> unsigned int i;
> hwaddr mem_base, mem_len;
> - char **node_path;
> Error *err = NULL;
>
> if (binfo->dtb_filename) {
> @@ -586,7 +586,6 @@ int arm_load_dtb(hwaddr addr, const struct arm_boot_info
> *binfo,
> }
> n++;
> }
> - g_strfreev(node_path);
>
> /*
> * We drop all the memory nodes which correspond to empty NUMA nodes
> --
> 2.47.2
>
>
