From: Marc-André Lureau Date: Mon, 27 Apr 2026 12:05:25 +0000 (+0400) Subject: hw/arm/aspeed: free fmc_model and spi_model on finalization X-Git-Tag: v11.1.0-rc0~124^2~6 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=be1ea14295c058e0ac8aeb0097ebde23f80397bd;p=thirdparty%2Fqemu.git hw/arm/aspeed: free fmc_model and spi_model on finalization The fmc_model and spi_model strings are allocated via g_strdup in property setters but never freed when the machine object is destroyed. Fixes: 9820e52fbef7 ("hw/arm/aspeed: Add machine properties to define the flash models") Reviewed-by: Cédric Le Goater Signed-off-by: Marc-André Lureau --- diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c index a3db3406c5f..a48c4420583 100644 --- a/hw/arm/aspeed.c +++ b/hw/arm/aspeed.c @@ -419,12 +419,21 @@ static void aspeed_machine_class_init(ObjectClass *oc, const void *data) aspeed_machine_class_props_init(oc); } +static void aspeed_machine_instance_finalize(Object *obj) +{ + AspeedMachineState *bmc = ASPEED_MACHINE(obj); + + g_free(bmc->fmc_model); + g_free(bmc->spi_model); +} + static const TypeInfo aspeed_machine_types[] = { { .name = TYPE_ASPEED_MACHINE, .parent = TYPE_MACHINE, .instance_size = sizeof(AspeedMachineState), .instance_init = aspeed_machine_instance_init, + .instance_finalize = aspeed_machine_instance_finalize, .class_size = sizeof(AspeedMachineClass), .class_init = aspeed_machine_class_init, .abstract = true,