BIER coverity fix in route downlaod

Change-Id: I9341f1554d804dfeeecef656e59e8598704863dc
Signed-off-by: Neale Ranns <nranns@cisco.com>

diff --git a/src/vnet/bier/bier.api b/src/vnet/bier/bier.api
index 6f88732..c7e9a98 100644 (file)
--- a/src/vnet/bier/bier.api
+++ b/src/vnet/bier/bier.api
@@ -109,7 +109,7 @@ autoreply define bier_route_add_del
 {
   u32 client_index;
   u32 context;
-  u16 br_bp;
+  u32 br_bp;
   u8 br_is_add;
   u8 br_is_replace;
   vl_api_bier_table_id_t br_tbl_id;

diff --git a/src/vnet/bier/bier_api.c b/src/vnet/bier/bier_api.c
index 7d541c4..9271258 100644 (file)
--- a/src/vnet/bier/bier_api.c
+++ b/src/vnet/bier/bier_api.c
@@ -162,10 +162,10 @@ vl_api_bier_route_add_del_t_handler (vl_api_bier_route_add_del_t * mp)
     vnm = vnet_get_main ();
     vnm->api_errno = 0;
 
-    bp = ntohs(mp->br_bp);
+    bp = ntohl(mp->br_bp);
     brpaths = NULL;
 
-    if (0 == bp || bp > 0xffff)
+    if (0 == bp || bp > BIER_BP_MAX)
     {
         rv = -1;
         goto done;