qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH v3 11/26] vnc: Plug minor memory leak in vnc_display_open()


From: Daniel P . Berrangé
Subject: Re: [PATCH v3 11/26] vnc: Plug minor memory leak in vnc_display_open()
Date: Tue, 30 Jun 2020 10:19:52 +0100
User-agent: Mutt/1.14.3 (2020-06-14)

On Tue, Jun 30, 2020 at 11:03:36AM +0200, Markus Armbruster wrote:
> vnc_display_print_local_addr() leaks the Error object when
> qio_channel_socket_get_local_address() fails.  Seems unlikely.  Called
> when we create a VNC display with vnc_display_open().  Plug the leak
> by passing NULL to ignore the error.
> 
> Cc: Daniel P. Berrange <berrange@redhat.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  ui/vnc.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




reply via email to

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