summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
author/C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org </C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org>2008-01-17 17:45:25 +0000
committer/C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org </C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org>2008-01-17 17:45:25 +0000
commit287c0f46e3499404d8e3bc35f7ae53f8fb678a1f (patch)
treebebc00d3c3c385e202e6418a48c314e7586e0a88
parentf77677c542c1b42d6a76cd114ae8f2ea6b07641e (diff)
Max Kellermann <max@duempel.org>:
fix double free() bug in the error output path of mcast_create()
-rw-r--r--ChangeLog1
-rw-r--r--src/mcast.c2
2 files changed, 1 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index 01dcc05..2d6cc3e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -84,6 +84,7 @@ o fix memory leaks in several error output paths
o use size_t for buffer sizes
o import only required C headers and put local headers on top to check
o remove superfluous initialization
+o fix double free() bug in the error output path of mcast_create()
version 0.9.5 (2007/07/29)
------------------------------
diff --git a/src/mcast.c b/src/mcast.c
index 414f031..e977c0b 100644
--- a/src/mcast.c
+++ b/src/mcast.c
@@ -152,7 +152,6 @@ __mcast_client_create_ipv4(struct mcast_sock *m, struct mcast_conf *conf)
sizeof(struct in_addr)) == -1) {
debug("mcast_sock_client_create:setsockopt3");
close(m->fd);
- free(m);
return -1;
}
@@ -182,7 +181,6 @@ __mcast_client_create_ipv6(struct mcast_sock *m, struct mcast_conf *conf)
sizeof(struct in_addr)) == -1) {
debug("mcast_sock_client_create:setsockopt3");
close(m->fd);
- free(m);
return -1;
}