Re: [Qemu-devel] [PATCH] qemu-sockets: use qapi_free_SocketAddress in cleanup

2016-07-08 Thread Paolo Bonzini
On 06/07/2016 18:42, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Commit 74b6ce43e3 uses the wrong free API for a SocketAddress, that > may leak some linked data. > > Signed-off-by: Marc-André Lureau > --- > util/qemu-sockets.c | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [Qemu-devel] [PATCH] qemu-sockets: use qapi_free_SocketAddress in cleanup

2016-07-08 Thread Shannon Zhao
On 2016/7/7 0:42, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Commit 74b6ce43e3 uses the wrong free API for a SocketAddress, that > may leak some linked data. > > Signed-off-by: Marc-André Lureau I'm planing to send a patch, then I find this :) Reviewed-by: Shannon Zhao

Re: [Qemu-devel] [PATCH] qemu-sockets: use qapi_free_SocketAddress in cleanup

2016-07-06 Thread Eric Blake
On 07/06/2016 10:42 AM, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Commit 74b6ce43e3 uses the wrong free API for a SocketAddress, that > may leak some linked data. > > Signed-off-by: Marc-André Lureau > --- > util/qemu-sockets.c | 2 +- > 1 file changed, 1 insertion(+), 1

[Qemu-devel] [PATCH] qemu-sockets: use qapi_free_SocketAddress in cleanup

2016-07-06 Thread marcandre . lureau
From: Marc-André Lureau Commit 74b6ce43e3 uses the wrong free API for a SocketAddress, that may leak some linked data. Signed-off-by: Marc-André Lureau --- util/qemu-sockets.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c index fb