diff options
| author |   Eyal Bari <ebari@cisco.com> | 2018-07-22 12:45:15 +0300 |
|---|---|---|
| committer |   Florin Coras <florin.coras@gmail.com> | 2018-07-24 15:15:25 +0000 |
| commit | 456ded496f93eb8156dc6e7b5a81d4a3aa604f60 (patch) | |
| tree | 426faba1e7096e963e3d36c61d19c0fea9175205 /src/vnet/mpls | |
| parent | c16a23c596169e76ee2b5091e2605c543659393d (diff) | |
fix vector index range checks (VPP-1353)
Change-Id: I63c36644c9d93f2c3ec6606ca0205b407499de4e
Signed-off-by: Eyal Bari <ebari@cisco.com>
(cherry picked from commit cd30774fa9280736ffaea3e9a51948593e8eebc2)
Signed-off-by: John Lo <loj@cisco.com>
Diffstat (limited to 'src/vnet/mpls')
| -rw-r--r-- | src/vnet/mpls/interface.c | 2 | ||||
| -rw-r--r-- | src/vnet/mpls/mpls_tunnel.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/vnet/mpls/interface.c b/src/vnet/mpls/interface.c index c792d56b12e..ec541f760de 100644 --- a/src/vnet/mpls/interface.c +++ b/src/vnet/mpls/interface.c @@ -29,7 +29,7 @@ mpls_sw_interface_is_enabled (u32 sw_if_index) { mpls_main_t * mm = &mpls_main; - if (vec_len(mm->mpls_enabled_by_sw_if_index) < sw_if_index) + if (vec_len(mm->mpls_enabled_by_sw_if_index) <= sw_if_index) return (0); return (mm->mpls_enabled_by_sw_if_index[sw_if_index]); diff --git a/src/vnet/mpls/mpls_tunnel.c b/src/vnet/mpls/mpls_tunnel.c index a142edf3f0a..84d569bcfee 100644 --- a/src/vnet/mpls/mpls_tunnel.c +++ b/src/vnet/mpls/mpls_tunnel.c @@ -52,7 +52,7 @@ static const char *mpls_tunnel_attribute_names[] = MPLS_TUNNEL_ATTRIBUTES; static mpls_tunnel_t* mpls_tunnel_get_from_sw_if_index (u32 sw_if_index) { - if ((vec_len(mpls_tunnel_db) < sw_if_index) || + if ((vec_len(mpls_tunnel_db) <= sw_if_index) || (~0 == mpls_tunnel_db[sw_if_index])) return (NULL); |