commit 39771adc1da61943978c18b58b35dedc9dddc4b0
author Florin Coras <fcoras@cisco.com> Mon Jul 27 14:42:10 2020 -0700
committer Dave Barach <openvpp@barachs.net> Tue Jul 28 12:17:04 2020 +0000
tree 6176263ec47d56be4f3d2b7d9391eb4eee94ac88
parent 508dc51bd075f6bb16862265c0c43e8efb76349c

lisp: fix cli locator-set name null termination

Type: fix

Change-Id: I5f550bd6a03f47b829ef99803cb6b9ac86329450
Signed-off-by: Florin Coras <fcoras@cisco.com>

src/vnet/lisp-cp/control.c

diff --git a/src/vnet/lisp-cp/control.c b/src/vnet/lisp-cp/control.c
index 4dc6836..cc3963d 100644
--- a/src/vnet/lisp-cp/control.c
+++ b/src/vnet/lisp-cp/control.c
@@ -2083,14 +2083,9 @@
 	      ls->name = vec_dup (a->name);
 
 	      if (!lcm->locator_set_index_by_name)
-		lcm->locator_set_index_by_name = hash_create_vec (
-								   /* size */
-								   0,
-								   sizeof
-								   (ls->name
-								    [0]),
-								   sizeof
-								   (uword));
+		lcm->locator_set_index_by_name =
+		  hash_create_vec ( /* size */ 0, sizeof (ls->name[0]),
+				   sizeof (uword));
 	      hash_set_mem (lcm->locator_set_index_by_name, ls->name,
 			    ls_index);
 

src/vnet/lisp-cp/lisp_cli.c

diff --git a/src/vnet/lisp-cp/lisp_cli.c b/src/vnet/lisp-cp/lisp_cli.c
index 31b2ca8..817fb50 100644
--- a/src/vnet/lisp-cp/lisp_cli.c
+++ b/src/vnet/lisp-cp/lisp_cli.c
@@ -1179,6 +1179,7 @@
 	}
     }
 
+  vec_terminate_c_string (locator_set_name);
   a->name = locator_set_name;
   a->locators = locators;
   a->is_add = is_add;