commit b06b35f0754747f9f178be155a2903b360aa2b6c
author Christian Franke <chris@opensourcerouting.org> Fri Dec 07 14:26:09 2012 +0000
committer David Lamparter <equinox@opensourcerouting.org> Mon Jan 14 16:09:19 2013 +0100
tree a92d70e724c5b5133ecebf9c4d4444bde3870cc2
parent 9499bf2bc6daf0a9b7170d3cf994daef2f1a8920

bgpd: fix a memleak on "set community none"

Signed-off-by: Christian Franke <chris@opensourcerouting.org>
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

bgpd/bgp_routemap.c

diff --git a/bgpd/bgp_routemap.c b/bgpd/bgp_routemap.c
index f220400..40f2076 100644
--- a/bgpd/bgp_routemap.c
+++ b/bgpd/bgp_routemap.c
@@ -1347,6 +1347,9 @@
 	{
 	  attr->flag &= ~(ATTR_FLAG_BIT (BGP_ATTR_COMMUNITIES));
 	  attr->community = NULL;
+	  /* See the longer comment down below. */
+	  if (old && old->refcnt == 0)
+	    community_free(old);
 	  return RMAP_OKAY;
 	}