aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeale Ranns <nranns@cisco.com>2017-12-19 01:34:30 -0800
committerChris Luke <chris_luke@comcast.com>2017-12-19 18:43:59 +0000
commitbe302d72031a7620fd1e79a620a99e8822bbac28 (patch)
tree914838d7bf9d4d476eaf10201fba711982bf1fd8
parent4bc2d7ab3990aa8c819cd8134d029aa0d5e5b0d7 (diff)
BIER coverity fix in route downlaod
Change-Id: I9341f1554d804dfeeecef656e59e8598704863dc Signed-off-by: Neale Ranns <nranns@cisco.com>
-rw-r--r--src/vnet/bier/bier.api2
-rw-r--r--src/vnet/bier/bier_api.c4
2 files changed, 3 insertions, 3 deletions
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;