aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFilip Tehlar <ftehlar@cisco.com>2016-08-11 14:43:19 +0200
committerFilip Tehlar <ftehlar@cisco.com>2016-08-11 14:43:19 +0200
commit77ae107ad647f25d22471fabdcf4b31097b1789c (patch)
tree5ff72797b17a5076d9eb83a3c3bfc443a1b579bc
parente262ff10d4d666ce22be199b020bde5f3fc42bf1 (diff)
VPP-321: Fix wrong checking of LISP mapping local flag
Change-Id: Ia1d5a53742a5ebe34566de0a68c2d0a81e002ffa Signed-off-by: Filip Tehlar <ftehlar@cisco.com>
-rw-r--r--vpp/vpp-api/api.c34
1 files changed, 14 insertions, 20 deletions
diff --git a/vpp/vpp-api/api.c b/vpp/vpp-api/api.c
index 18063acc7d5..db9cd203060 100644
--- a/vpp/vpp-api/api.c
+++ b/vpp/vpp-api/api.c
@@ -5662,32 +5662,25 @@ send_lisp_eid_table_details (mapping_t * mapit,
u32 context, u8 filter)
{
vl_api_lisp_eid_table_details_t *rmp = NULL;
- lisp_cp_main_t *lcm = vnet_lisp_cp_get_main ();
- locator_set_t *ls = NULL;
gid_address_t *gid = NULL;
u8 *mac = 0;
ip_prefix_t *ip_prefix = NULL;
- ls = pool_elt_at_index (lcm->locator_set_pool, mapit->locator_set_index);
-
switch (filter)
{
- case 0:
- break;
- case 1:
- if (!ls->local)
- {
- return;
- }
- break;
- case 2:
- if (ls->local)
- {
- return;
- }
+ case 0: /* all mappings */
break;
+
+ case 1: /* local only */
+ if (!mapit->local)
+ return;
+
+ case 2: /* remote only */
+ if (mapit->local)
+ return;
+
default:
- clib_warning ("Filter error, unknown filter: %d\n", filter);
+ clib_warning ("Filter error, unknown filter: %d", filter);
return;
}
@@ -5699,7 +5692,7 @@ send_lisp_eid_table_details (mapping_t * mapit,
memset (rmp, 0, sizeof (*rmp));
rmp->_vl_msg_id = ntohs (VL_API_LISP_EID_TABLE_DETAILS);
rmp->locator_set_index = mapit->locator_set_index;
- rmp->is_local = ls->local;
+ rmp->is_local = mapit->local;
rmp->ttl = mapit->ttl;
rmp->authoritative = mapit->authoritative;
@@ -5760,7 +5753,8 @@ vl_api_lisp_eid_table_dump_t_handler (vl_api_lisp_eid_table_dump_t * mp)
return;
mapit = pool_elt_at_index (lcm->mapping_pool, mi);
- send_lisp_eid_table_details (mapit, q, mp->context, mp->filter);
+ send_lisp_eid_table_details (mapit, q, mp->context,
+ 0 /* ignore filter */);
}
else
{