summaryrefslogtreecommitdiffstats
path: root/src/vnet/qos
diff options
context:
space:
mode:
authorEyal Bari <ebari@cisco.com>2018-07-18 17:15:26 +0300
committerJohn Lo <loj@cisco.com>2018-07-18 21:17:58 +0000
commit692250bedfc047ec9f88e7308daf456ed5f4a2a7 (patch)
treec4c52d134b5a2289c01385c2d3c2ab0631527696 /src/vnet/qos
parente4b49c738b74ea25e190cac68399c724a023f067 (diff)
qos mark/record: fix disable-sw_if_index range check
Change-Id: I2abb3ceebae3a32cee9aa2a999bd47c37719d3ac Signed-off-by: Eyal Bari <ebari@cisco.com> Signed-off-by: Igor Mikhailov (imichail) <imichail@cisco.com> (cherry picked from commit 07f3586c5a9868ccee381af528de39b373d7720a)
Diffstat (limited to 'src/vnet/qos')
-rw-r--r--src/vnet/qos/qos_mark.c2
-rw-r--r--src/vnet/qos/qos_record.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/src/vnet/qos/qos_mark.c b/src/vnet/qos/qos_mark.c
index cec7192b018..5dc87e8ef81 100644
--- a/src/vnet/qos/qos_mark.c
+++ b/src/vnet/qos/qos_mark.c
@@ -361,7 +361,7 @@ qos_mark_enable (u32 sw_if_index,
int
qos_mark_disable (u32 sw_if_index, qos_source_t output_source)
{
- if (vec_len (qos_mark_configs[output_source]) < sw_if_index)
+ if (vec_len (qos_mark_configs[output_source]) <= sw_if_index)
return VNET_API_ERROR_NO_MATCHING_INTERFACE;
if (INDEX_INVALID == qos_mark_configs[output_source][sw_if_index])
return VNET_API_ERROR_VALUE_EXIST;
diff --git a/src/vnet/qos/qos_record.c b/src/vnet/qos/qos_record.c
index c69b4f1f9d0..7181e3bf599 100644
--- a/src/vnet/qos/qos_record.c
+++ b/src/vnet/qos/qos_record.c
@@ -68,7 +68,7 @@ qos_record_enable (u32 sw_if_index, qos_source_t input_source)
int
qos_record_disable (u32 sw_if_index, qos_source_t input_source)
{
- if (vec_len (qos_record_configs[input_source]) < sw_if_index)
+ if (vec_len (qos_record_configs[input_source]) <= sw_if_index)
return VNET_API_ERROR_NO_MATCHING_INTERFACE;
if (0 == qos_record_configs[input_source][sw_if_index])