Re: [PATCH-for-5.1 4/4] hw/avr/boot: Fix memory leak in avr_load_firmware()

2020-07-20 Thread Philippe Mathieu-Daudé
On 7/20/20 4:18 PM, Daniel P. Berrangé wrote: > On Tue, Jul 14, 2020 at 06:42:57PM +0200, 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. >

Re: [PATCH-for-5.1 4/4] hw/avr/boot: Fix memory leak in avr_load_firmware()

2020-07-20 Thread Daniel P . Berrangé
On Tue, Jul 14, 2020 at 06:42:57PM +0200, 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) > F

Re: [PATCH-for-5.1 4/4] hw/avr/boot: Fix memory leak in avr_load_firmware()

2020-07-20 Thread Philippe Mathieu-Daudé
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: 7dd8f6fde

[PATCH-for-5.1 4/4] hw/avr/boot: Fix memory leak in avr_load_firmware()

2020-07-14 Thread Philippe Mathieu-Daudé
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: P