[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH-for-5.1 4/4] hw/avr/boot: Fix memory leak in avr_load_firmwar
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH-for-5.1 4/4] hw/avr/boot: Fix memory leak in avr_load_firmware() |
Date: |
Mon, 20 Jul 2020 14:39:09 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
ping?
On 7/14/20 6:42 PM, Philippe Mathieu-Daudé wrote:
> The value returned by qemu_find_file() must be freed.
>
> This fixes Coverity issue CID 1430449, which points out
> that the memory returned by qemu_find_file() is leaked.
>
> Fixes: Coverity CID 1430449 (RESOURCE_LEAK)
> Fixes: 7dd8f6fde4 ('hw/avr: Add support for loading ELF/raw binaries')
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> hw/avr/boot.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/avr/boot.c b/hw/avr/boot.c
> index 6fbcde4061..151734f82d 100644
> --- a/hw/avr/boot.c
> +++ b/hw/avr/boot.c
> @@ -60,7 +60,7 @@ static const char *avr_elf_e_flags_to_cpu_type(uint32_t
> flags)
> bool avr_load_firmware(AVRCPU *cpu, MachineState *ms,
> MemoryRegion *program_mr, const char *firmware)
> {
> - const char *filename;
> + g_autofree char *filename;
> int bytes_loaded;
> uint64_t entry;
> uint32_t e_flags;
>
- [PATCH-for-5.1 0/4] misc: Document qemu_find_file and fix memory leak in avr_load_firmware, Philippe Mathieu-Daudé, 2020/07/14
- [PATCH-for-5.1 1/4] qemu/osdep: Document os_find_datadir() return value, Philippe Mathieu-Daudé, 2020/07/14
- [PATCH-for-5.1 2/4] qemu/osdep: Reword qemu_get_exec_dir() documentation, Philippe Mathieu-Daudé, 2020/07/14
- [PATCH-for-5.1 4/4] hw/avr/boot: Fix memory leak in avr_load_firmware(), Philippe Mathieu-Daudé, 2020/07/14
- [PATCH-for-5.1 3/4] qemu-common: Document qemu_find_file(), Philippe Mathieu-Daudé, 2020/07/14
- Re: [PATCH-for-5.1 0/4] misc: Document qemu_find_file and fix memory leak in avr_load_firmware, Michael Rolnik, 2020/07/20
- Re: [PATCH-for-5.1 0/4] misc: Document qemu_find_file and fix memory leak in avr_load_firmware, Philippe Mathieu-Daudé, 2020/07/20