qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] vnc: remove bogus object_unref on client socket


From: Bandan Das
Subject: Re: [Qemu-devel] [PATCH] vnc: remove bogus object_unref on client socket
Date: Thu, 15 Feb 2018 13:24:14 -0500
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)

Daniel P. Berrangé <address@hidden> writes:

> vnc_listen_io() does not own the reference on the 'cioc' parameter is it
> passed, so should not be unref'ing it.
>
> Reported-by: Bandan Das <address@hidden>
> Signed-off-by: Daniel P. Berrangé <address@hidden>

Daniel, wouldn't a Fixes: 13e1d0e71e78a925848258391a6e616b6b5ae219
be helpful here ?

> ---
>  ui/vnc.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/ui/vnc.c b/ui/vnc.c
> index c715bae1cf..b97769aa9e 100644
> --- a/ui/vnc.c
> +++ b/ui/vnc.c
> @@ -3152,7 +3152,6 @@ static void vnc_listen_io(QIONetListener *listener,
>                           isWebsock ? "vnc-ws-server" : "vnc-server");
>      qio_channel_set_delay(QIO_CHANNEL(cioc), false);
>      vnc_connect(vd, cioc, false, isWebsock);
> -    object_unref(OBJECT(cioc));
>  }
>  
>  static const DisplayChangeListenerOps dcl_ops = {



reply via email to

[Prev in Thread] Current Thread [Next in Thread]