Re: [Qemu-devel] [PATCH] l2tpv3: fix possible double free

2014-11-16 Thread zhanghailiang
On 2014/11/14 16:45, Paolo Bonzini wrote: On 14/11/2014 02:39, zhanghailiang wrote: freeaddrinfo(result) does not assign result = NULL, after frees it. There will be a double free when it goes error case. It is reported by covertiy. Signed-off-by: zhanghailiang --- net/l2tpv3.c | 1 + 1 f

Re: [Qemu-devel] [PATCH] l2tpv3: fix possible double free

2014-11-14 Thread Paolo Bonzini
On 14/11/2014 02:39, zhanghailiang wrote: > freeaddrinfo(result) does not assign result = NULL, after frees it. > There will be a double free when it goes error case. > It is reported by covertiy. > > Signed-off-by: zhanghailiang > --- > net/l2tpv3.c | 1 + > 1 file changed, 1 insertion(+) >

Re: [Qemu-devel] [PATCH] l2tpv3: fix possible double free

2014-11-13 Thread Gonglei
On 2014/11/14 9:39, zhanghailiang wrote: > freeaddrinfo(result) does not assign result = NULL, after frees it. > There will be a double free when it goes error case. > It is reported by covertiy. > > Signed-off-by: zhanghailiang > --- > net/l2tpv3.c | 1 + > 1 file changed, 1 insertion(+) Rev

[Qemu-devel] [PATCH] l2tpv3: fix possible double free

2014-11-13 Thread zhanghailiang
freeaddrinfo(result) does not assign result = NULL, after frees it. There will be a double free when it goes error case. It is reported by covertiy. Signed-off-by: zhanghailiang --- net/l2tpv3.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/l2tpv3.c b/net/l2tpv3.c index 528d95b..f9e0c9