Re: [libvirt] [PATCH v3 2/3] conf: add check if migration_host is a localhost address

2014-10-15 Thread Chen, Fan
On Wed, 2014-10-15 at 04:46 -0400, John Ferlan wrote: > This patch has triggered a Coverity RESOURCE_LEAK (3 actually) Right, I will make a patch to fix it. Thank you for catching that. > > On 10/08/2014 09:54 PM, Chen, Fan wrote: > > On Wed, 2014-10-08 at 12:33 +0200, Ján Tomko wrote: > >> O

Re: [libvirt] [PATCH v3 2/3] conf: add check if migration_host is a localhost address

2014-10-15 Thread John Ferlan
This patch has triggered a Coverity RESOURCE_LEAK (3 actually) On 10/08/2014 09:54 PM, Chen, Fan wrote: > On Wed, 2014-10-08 at 12:33 +0200, Ján Tomko wrote: >> On 10/07/2014 06:07 AM, Chen Fan wrote: >>> Signed-off-by: Chen Fan >> diff --git a/src/util/virsocketaddr.c b/src/util/virsocketaddr.

Re: [libvirt] [PATCH v3 2/3] conf: add check if migration_host is a localhost address

2014-10-08 Thread Chen, Fan
On Wed, 2014-10-08 at 12:33 +0200, Ján Tomko wrote: > On 10/07/2014 06:07 AM, Chen Fan wrote: > > Signed-off-by: Chen Fan > > --- > > src/libvirt_private.syms | 1 + > > src/qemu/qemu_conf.c | 50 > > > > src/qemu/qemu_conf.h | 2 ++ >

Re: [libvirt] [PATCH v3 2/3] conf: add check if migration_host is a localhost address

2014-10-08 Thread Ján Tomko
On 10/07/2014 06:07 AM, Chen Fan wrote: > Signed-off-by: Chen Fan > --- > src/libvirt_private.syms | 1 + > src/qemu/qemu_conf.c | 50 > > src/qemu/qemu_conf.h | 2 ++ > src/util/virsocketaddr.c | 24 +++ > src/util/

[libvirt] [PATCH v3 2/3] conf: add check if migration_host is a localhost address

2014-10-06 Thread Chen Fan
Signed-off-by: Chen Fan --- src/libvirt_private.syms | 1 + src/qemu/qemu_conf.c | 50 src/qemu/qemu_conf.h | 2 ++ src/util/virsocketaddr.c | 24 +++ src/util/virsocketaddr.h | 2 ++ 5 files changed, 79 insertions(+