[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 20/37] char: free MuxDriver when closing
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2 20/37] char: free MuxDriver when closing |
Date: |
Mon, 1 Aug 2016 14:50:06 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 |
On 28/07/2016 16:37, address@hidden wrote:
> From: Marc-André Lureau <address@hidden>
>
> Similarly to other chr_close callbacks, free char type specific data.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> qemu-char.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/qemu-char.c b/qemu-char.c
> index 6ed6dd6..e276485 100644
> --- a/qemu-char.c
> +++ b/qemu-char.c
> @@ -786,6 +786,14 @@ static GSource *mux_chr_add_watch(CharDriverState *s,
> GIOCondition cond)
> return d->drv->chr_add_watch(d->drv, cond);
> }
>
> +static void mux_chr_close(struct CharDriverState *chr)
> +{
> + MuxDriver *d = chr->opaque;
> +
> + /* is more cleanup needed? */
Perhaps qemu_chr_free(d->drv)?
Paolo
> + g_free(d);
> +}
> +
> static CharDriverState *qemu_chr_open_mux(const char *id,
> ChardevBackend *backend,
> ChardevReturn *ret, Error **errp)
> @@ -810,6 +818,7 @@ static CharDriverState *qemu_chr_open_mux(const char *id,
> chr->opaque = d;
> d->drv = drv;
> d->focus = -1;
> + chr->chr_close = mux_chr_close;
> chr->chr_write = mux_chr_write;
> chr->chr_update_read_handler = mux_chr_update_read_handler;
> chr->chr_accept_input = mux_chr_accept_input;
>
- Re: [Qemu-devel] [PATCH v2 20/37] char: free MuxDriver when closing,
Paolo Bonzini <=