Re: [PATCH-for-7.0] target/i386/hvf: Free ressources when vCPU is destroyed

[Igor Mammedov]

On Tue, 22 Mar 2022 20:07:45 +0100
Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com> wrote:

> From: Philippe Mathieu-Daudé <f4bug@amsat.org>
> 
> Both xsave_buf and hvf_caps are allocated in hvf_arch_init_vcpu(),
> free them in hvf_arch_vcpu_destroy().
> 
> Reported-by: Mark Kanda <mark.kanda@oracle.com>
> Suggested-by: Igor Mammedov <imammedo@redhat.com>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Reviewed-by: Igor Mammedov <imammedo@redhat.com>

> ---
>  target/i386/hvf/hvf.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/target/i386/hvf/hvf.c b/target/i386/hvf/hvf.c
> index fc12c02fb2..39fa4641b9 100644
> --- a/target/i386/hvf/hvf.c
> +++ b/target/i386/hvf/hvf.c
> @@ -163,7 +163,9 @@ void hvf_arch_vcpu_destroy(CPUState *cpu)
>      X86CPU *x86_cpu = X86_CPU(cpu);
>      CPUX86State *env = &x86_cpu->env;
>  
> +    g_free(env->xsave_buf);
>      g_free(env->hvf_mmio_buf);
> +    g_free(hvf_state->hvf_caps);
>  }
>  
>  static void init_tsc_freq(CPUX86State *env)