commit e2a9258c16bce1b3797efb9be354d20d68236194
author Denil Vira <denil@cumulusnetworks.com> Tue Aug 11 13:34:59 2015 -0700
committer Paul Jakma <paul@quagga.net> Thu Sep 24 15:26:43 2015 +0100
tree f08790964b068afe0724518774dfe315a75200ca
parent 57cd5e7a2623258e1a095d63e0faaad2cddf61e8

bgpd: Fix memory leak in bgpd/bgp_route.c

In function bgp_aggregate_add, variables 'aspath' and 'community'
are malloced but not guaranteed to be freed before the function
returns.

Signed-off-by: Denil Vira <denil@cumulusnetworks.com>

bgpd/bgp_route.c

diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c
index 7de9461..4db2e3e 100644
--- a/bgpd/bgp_route.c
+++ b/bgpd/bgp_route.c
@@ -4888,6 +4888,13 @@
       /* Process change. */
       bgp_process (bgp, rn, afi, safi);
     }
+  else
+    {
+      if (aspath)
+	aspath_free (aspath);
+      if (community)
+	community_free (community);
+    }
 }
 
 void