Re: [Qemu-devel] [PATCH] qbus: fix memory leak in qbus_free()

[Blue Swirl]

Thanks, applied.

On Thu, May 27, 2010 at 5:35 AM, Isaku Yamahata <address@hidden> wrote:
> BusState::name is allocated in qbus_create_inplace().
> So it should be freed by qbus_free().
>
> Signed-off-by: Isaku Yamahata <address@hidden>
> ---
>  hw/qdev.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/hw/qdev.c b/hw/qdev.c
> index af17486..2845af5 100644
> --- a/hw/qdev.c
> +++ b/hw/qdev.c
> @@ -700,6 +700,7 @@ void qbus_free(BusState *bus)
>         QLIST_REMOVE(bus, sibling);
>         bus->parent->num_child_bus--;
>     }
> +    qemu_free((void*)bus->name);
>     if (bus->qdev_allocated) {
>         qemu_free(bus);
>     }
> --
> 1.6.6.1
>
>