[Qemu-devel] [PATCH v2] hw/arm/realview.c: Fix memory leak in realview_init()

2014-11-21 Thread Nikita Belov
(realview.c:55) ==16879==by 0x28988C: realview_pb_a8_init (realview.c:375) ==16879==by 0x341426: main (vl.c:4413) Signed-off-by: Nikita Belov zod...@ispras.ru --- hw/arm/realview.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/arm/realview.c b/hw/arm/realview.c index

Re: [Qemu-devel] [PATCH] hw/arm/realview.c: Fix memory leak in realview_init()

2014-11-19 Thread Nikita Belov
On 2014-10-29 17:03, Nikita Belov wrote: Variable 'ram_lo' is allocated unconditionally, but used only in some cases. When it is unused pointer will be lost at function exit, resulting in a memory leak. Free memory in this case. Valgrind output: ==16879== 240 bytes in 1 blocks are definitely

Re: [Qemu-devel] [PATCH] hw/arm/realview.c: Fix memory leak in realview_init()

2014-10-31 Thread Nikita Belov
On 2014-10-29 19:03, Peter Maydell wrote: On 29 October 2014 14:03, Nikita Belov zod...@ispras.ru wrote: Variable 'ram_lo' is allocated unconditionally, but used only in some cases. When it is unused pointer will be lost at function exit, resulting in a memory leak. Free memory in this case

[Qemu-devel] [PATCH] hw/arm/realview.c: Fix memory leak in realview_init()

2014-10-29 Thread Nikita Belov
==by 0x28988C: realview_pb_a8_init (realview.c:375) ==16879==by 0x341426: main (vl.c:4413) Signed-off-by: Nikita Belov zod...@ispras.ru --- hw/arm/realview.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/arm/realview.c b/hw/arm/realview.c index af65aa4..673a540 100644 --- a/hw/arm

[Qemu-devel] [PATCH] hw/i386/acpi-build.c: Fix memory leak in acpi_build_tables_cleanup()

2014-10-29 Thread Nikita Belov
) ==23931==by 0x2EEC3C: main (vl.c:4504) Signed-off-by: Nikita Belov zod...@ispras.ru --- hw/i386/acpi-build.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c index 00be4bb..c1778db 100644 --- a/hw/i386/acpi-build.c +++ b/hw