From: Neale Ranns <nranns@cisco.com>
Date: Tue, 19 Dec 2017 09:34:30 +0000 (-0800)
Subject: BIER coverity fix in route downlaod
X-Git-Tag: v18.04-rc0~13
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;ds=sidebyside;h=be302d72031a7620fd1e79a620a99e8822bbac28;p=vpp.git

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 6f88732aba9..c7e9a98209b 100644
--- 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 7d541c49182..92712580f14 100644
--- 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;