[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 06/13] contrib/elf2dmp: Always destroy PA space
From: |
Akihiko Odaki |
Subject: |
[PATCH v2 06/13] contrib/elf2dmp: Always destroy PA space |
Date: |
Tue, 05 Mar 2024 16:36:23 +0900 |
Destroy PA space even if paging base couldn't be found, fixing memory
leak.
Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
---
contrib/elf2dmp/main.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/contrib/elf2dmp/main.c b/contrib/elf2dmp/main.c
index 140ac6e00cfe..25cf0fdff724 100644
--- a/contrib/elf2dmp/main.c
+++ b/contrib/elf2dmp/main.c
@@ -550,7 +550,7 @@ int main(int argc, char *argv[])
va_space_create(&vs, &ps, state->cr[3]);
if (!fix_dtb(&vs, &qemu_elf)) {
eprintf("Failed to find paging base\n");
- goto out_elf;
+ goto out_ps;
}
printf("CPU #0 IDT is at 0x%016"PRIx64"\n", state->idt.base);
@@ -635,7 +635,6 @@ out_pdb_file:
unlink(PDB_NAME);
out_ps:
pa_space_destroy(&ps);
-out_elf:
QEMU_Elf_exit(&qemu_elf);
return err;
--
2.44.0
- [PATCH v2 08/13] contrib/elf2dmp: Use lduw_le_p() to read PDB, (continued)
- [PATCH v2 08/13] contrib/elf2dmp: Use lduw_le_p() to read PDB, Akihiko Odaki, 2024/03/05
- [PATCH v2 13/13] contrib/elf2dmp: Clamp QEMU note to file size, Akihiko Odaki, 2024/03/05
- [PATCH v2 07/13] contrib/elf2dmp: Ensure segment fits in file, Akihiko Odaki, 2024/03/05
- [PATCH v2 09/13] contrib/elf2dmp: Use rol64() to decode, Akihiko Odaki, 2024/03/05
- [PATCH v2 11/13] contrib/elf2dmp: Build only for little endian host, Akihiko Odaki, 2024/03/05
- [PATCH v2 06/13] contrib/elf2dmp: Always destroy PA space,
Akihiko Odaki <=
- [PATCH v2 10/13] MAINTAINERS: Add Akihiko Odaki as a elf2dmp reviewer, Akihiko Odaki, 2024/03/05