aboutsummaryrefslogtreecommitdiff
path: root/net
diff options
context:
space:
mode:
authorzhanghailiang <zhang.zhanghailiang@huawei.com>2014-11-14 09:39:23 +0800
committerPaolo Bonzini <pbonzini@redhat.com>2014-11-14 12:16:24 +0100
commit77374582ab961af2c5e702f767f52179d5f7676c (patch)
treec794bb5f0d62a00ddcc3a74ab40276698b34e0d2 /net
parent5bbebf622897a59db5da4c468e737bfec4d71280 (diff)
downloadqemu-77374582ab961af2c5e702f767f52179d5f7676c.zip
qemu-77374582ab961af2c5e702f767f52179d5f7676c.tar.gz
qemu-77374582ab961af2c5e702f767f52179d5f7676c.tar.bz2
l2tpv3: fix possible double free
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. Reviewed-by: Gonglei <arei.gonglei@huawei.com> Cc: qemu-stable@nongnu.org Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'net')
-rw-r--r--net/l2tpv3.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/net/l2tpv3.c b/net/l2tpv3.c
index 528d95b..65db5ef 100644
--- a/net/l2tpv3.c
+++ b/net/l2tpv3.c
@@ -660,7 +660,6 @@ int net_init_l2tpv3(const NetClientOptions *opts,
if (fd == -1) {
fd = -errno;
error_report("l2tpv3_open : socket creation failed, errno = %d", -fd);
- freeaddrinfo(result);
goto outerr;
}
if (bind(fd, (struct sockaddr *) result->ai_addr, result->ai_addrlen)) {