summaryrefslogtreecommitdiff
path: root/bgpd
diff options
context:
space:
mode:
authorPaul Jakma <paul.jakma@sun.com>2006-05-08 14:25:09 +0000
committerPaul Jakma <paul.jakma@sun.com>2006-05-08 14:25:09 +0000
commit542bcb724c410283d7a3f317c8fe5e8a0b58f196 (patch)
treecefe67e9b39990827c64fda884adeda208af9787 /bgpd
parent65ca75e0d1e7e440bf0d2a6a031d4148024c9d12 (diff)
[bgpd] CID#62 fix double-free, use-after-free in community_str2com
2006-05-08 Paul Jakma <paul.jakma@sun.com> * bgp_community.c: (community_str2com) Coverity CID#62, fix double-free, use-after-free.
Diffstat (limited to 'bgpd')
-rw-r--r--bgpd/ChangeLog5
-rw-r--r--bgpd/bgp_community.c2
2 files changed, 6 insertions, 1 deletions
diff --git a/bgpd/ChangeLog b/bgpd/ChangeLog
index 5599801e..16a56fc2 100644
--- a/bgpd/ChangeLog
+++ b/bgpd/ChangeLog
@@ -1,3 +1,8 @@
+2006-05-08 Paul Jakma <paul.jakma@sun.com>
+
+ * bgp_community.c: (community_str2com) Coverity CID#62, fix
+ double-free, use-after-free.
+
2006-05-04 Paul Jakma <paul.jakma@sun.com>
* (general) VPNv4 fixes. Certain VPNv4 code was not enabled.
diff --git a/bgpd/bgp_community.c b/bgpd/bgp_community.c
index b419a203..07b8cf81 100644
--- a/bgpd/bgp_community.c
+++ b/bgpd/bgp_community.c
@@ -596,7 +596,7 @@ community_str2com (const char *str)
default:
if (com)
community_free (com);
- break;
+ return NULL;
}
} while (str);