[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] hw/arm/armsse: Fix memory leak in error-exit pa
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH] hw/arm/armsse: Fix memory leak in error-exit path |
Date: |
Fri, 15 Feb 2019 21:21:25 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 2/15/19 12:37 PM, Peter Maydell wrote:
> Coverity points out (CID 1398632, CID 1398650) that we
> leak a couple of allocated strings in the error-exit
> code path for setting up the MHUs in the ARMSSE.
> Fix this bug by moving the allocate-and-free of each
> string to be closer to the use, so we do the free before
> doing the error-exit check.
>
> Fixes: f8574705f62b38a ("hw/arm/armsse: Add unimplemented-device stubs for
> MHUs")
> Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/arm/armsse.c | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/hw/arm/armsse.c b/hw/arm/armsse.c
> index 9a8c49547db..d0207dbabc7 100644
> --- a/hw/arm/armsse.c
> +++ b/hw/arm/armsse.c
> @@ -762,26 +762,28 @@ static void armsse_realize(DeviceState *dev, Error
> **errp)
>
> if (info->has_mhus) {
> for (i = 0; i < ARRAY_SIZE(s->mhu); i++) {
> - char *name = g_strdup_printf("MHU%d", i);
> - char *port = g_strdup_printf("port[%d]", i + 3);
> + char *name;
> + char *port;
>
> + name = g_strdup_printf("MHU%d", i);
> qdev_prop_set_string(DEVICE(&s->mhu[i]), "name", name);
> qdev_prop_set_uint64(DEVICE(&s->mhu[i]), "size", 0x1000);
> object_property_set_bool(OBJECT(&s->mhu[i]), true,
> "realized", &err);
> + g_free(name);
> if (err) {
> error_propagate(errp, err);
> return;
> }
> + port = g_strdup_printf("port[%d]", i + 3);
> mr = sysbus_mmio_get_region(SYS_BUS_DEVICE(&s->mhu[i]), 0);
> object_property_set_link(OBJECT(&s->apb_ppc0), OBJECT(mr),
> port, &err);
> + g_free(port);
> if (err) {
> error_propagate(errp, err);
> return;
> }
> - g_free(name);
> - g_free(port);
> }
> }
>
>