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:09:35 +0000 |
| commit | 07f3586c5a9868ccee381af528de39b373d7720a (patch) | |
| tree | 4042fcdc18879e9b8a09fd5891700a7928267332 /src/vnet/qos | |
| parent | 400378f056f18e9982684feff094258288ae748d (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>
Diffstat (limited to 'src/vnet/qos')
| -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]) |