[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] microvm: fix memory leak in microvm_fix_kernel_cmdline
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH] microvm: fix memory leak in microvm_fix_kernel_cmdline |
Date: |
Tue, 12 Nov 2019 18:59:38 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 12/11/19 17:34, Sergio Lopez wrote:
> In microvm_fix_kernel_cmdline(), fw_cfg_modify_string() is duplicating
> cmdline instead of taking ownership of it. Free it afterwards to avoid
> leaking it.
>
> Reported-by: Coverity (CID 1407218)
> Suggested-by: Peter Maydell <address@hidden>
> Signed-off-by: Sergio Lopez <address@hidden>
> ---
> hw/i386/microvm.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/hw/i386/microvm.c b/hw/i386/microvm.c
> index 8aacd6c8d1..def37e60f7 100644
> --- a/hw/i386/microvm.c
> +++ b/hw/i386/microvm.c
> @@ -331,6 +331,8 @@ static void microvm_fix_kernel_cmdline(MachineState
> *machine)
>
> fw_cfg_modify_i32(x86ms->fw_cfg, FW_CFG_CMDLINE_SIZE, strlen(cmdline) +
> 1);
> fw_cfg_modify_string(x86ms->fw_cfg, FW_CFG_CMDLINE_DATA, cmdline);
> +
> + g_free(cmdline);
> }
>
> static void microvm_machine_state_init(MachineState *machine)
>
Queued, thanks.
Paolo