commit d23d39cc012e5b1075a2323d6e940ae5ad43de46
author John Lo <loj@cisco.com> Thu Sep 13 15:08:08 2018 -0400
committer Damjan Marion <dmarion@me.com> Fri Sep 14 05:03:42 2018 +0000
tree 2f2c4f4ba593e08607ec8a0d1728b53983727934
parent 12a30202ec213d1ffbee503e61b1cec0bc4861bb

Fix memory leak in API/CLI to create/modify SR policies

The segment list vector built by the API/CLI is not freed after
SR policy creation or modification.

Change-Id: If439005481cada6c6af7cb560fe7a4381dd49384
Signed-off-by: John Lo <loj@cisco.com>

src/vnet/srmpls/sr_mpls_api.c

diff --git a/src/vnet/srmpls/sr_mpls_api.c b/src/vnet/srmpls/sr_mpls_api.c
index a6294e3..2c36c0d 100644
--- a/src/vnet/srmpls/sr_mpls_api.c
+++ b/src/vnet/srmpls/sr_mpls_api.c
@@ -67,6 +67,7 @@
   int rv = 0;
   rv = sr_mpls_policy_add (ntohl (mp->bsid),
 			   segments, mp->type, ntohl (mp->weight));
+  vec_free (segments);
 
   REPLY_MACRO (VL_API_SR_MPLS_POLICY_ADD_REPLY);
 }
@@ -91,6 +92,7 @@
   rv = sr_mpls_policy_mod (ntohl (mp->bsid),
 			   mp->operation, segments, ntohl (mp->sl_index),
 			   ntohl (mp->weight));
+  vec_free (segments);
 
   REPLY_MACRO (VL_API_SR_MPLS_POLICY_MOD_REPLY);
 }