[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 07/25] spapr: Plug minor memory leak in spapr_machine_init()
From: |
Markus Armbruster |
Subject: |
[PATCH v2 07/25] spapr: Plug minor memory leak in spapr_machine_init() |
Date: |
Wed, 24 Jun 2020 10:37:19 +0200 |
spapr_machine_init() leaks an Error object when
kvmppc_check_papr_resize_hpt() fails and spapr->resize_hpt is
SPAPR_RESIZE_HPT_DISABLED, i.e. when the host doesn't support hash
page table resizing, and the user didn't ask for it. As harmless as
memory leaks can possibly be. Plug it.
Fixes: 30f4b05bd090564181554d0890605eb2c143e4ea
Cc: David Gibson <dgibson@redhat.com>
Cc: qemu-ppc@nongnu.org
Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
hw/ppc/spapr.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
index bd9345cdac..9bd2183ead 100644
--- a/hw/ppc/spapr.c
+++ b/hw/ppc/spapr.c
@@ -2731,6 +2731,7 @@ static void spapr_machine_init(MachineState *machine)
error_report_err(resize_hpt_err);
exit(1);
}
+ error_free(resize_hpt_err);
spapr->rma_size = spapr_rma_size(spapr, &error_fatal);
--
2.26.2
- [PATCH v2 00/25] Error handling fixes & cleanups, Markus Armbruster, 2020/06/24
- [PATCH v2 07/25] spapr: Plug minor memory leak in spapr_machine_init(),
Markus Armbruster <=
- [PATCH v2 10/25] test-util-filemonitor: Plug unlikely memory leak, Markus Armbruster, 2020/06/24
- [PATCH v2 05/25] tests: Use error_free_or_abort() where appropriate, Markus Armbruster, 2020/06/24
- [PATCH v2 14/25] qdev: Drop qbus_set_hotplug_handler() parameter @errp, Markus Armbruster, 2020/06/24
- [PATCH v2 03/25] Clean up some calls to ignore Error objects the right way, Markus Armbruster, 2020/06/24
- [PATCH v2 04/25] tests: Use &error_abort where appropriate, Markus Armbruster, 2020/06/24