diff options
| author |   Eyal Bari <ebari@cisco.com> | 2018-07-18 17:15:26 +0300 |
|---|---|---|
| committer |   John Lo <loj@cisco.com> | 2018-07-18 21:17:58 +0000 |
| commit | 692250bedfc047ec9f88e7308daf456ed5f4a2a7 (patch) | |
| tree | c4c52d134b5a2289c01385c2d3c2ab0631527696 | |
| parent | e4b49c738b74ea25e190cac68399c724a023f067 (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)
| -rw-r--r-- | src/vnet/qos/qos_mark.c | 2 | ||||
| -rw-r--r-- | src/vnet/qos/qos_record.c | 2 |
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]) |