From: Chenmin Sun <chenmin.sun@intel.com>
Date: Mon, 29 Mar 2021 19:50:02 +0000 (+0800)
Subject: avf: fix coverity issues
X-Git-Tag: v21.10-rc0~316
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F92%2F31792%2F3;p=vpp.git

avf: fix coverity issues

CID #220403 ~ 220406

Type: fix

Signed-off-by: Chenmin Sun <chenmin.sun@intel.com>
Change-Id: Ibb65f9f2b17b9a9d45c077933b033ca3a65ae42f
---

diff --git a/src/plugins/avf/avf_fdir_lib.c b/src/plugins/avf/avf_fdir_lib.c
index 44c10dc7388..f38614e87ec 100644
--- a/src/plugins/avf/avf_fdir_lib.c
+++ b/src/plugins/avf/avf_fdir_lib.c
@@ -73,7 +73,7 @@ avf_fdir_rcfg_set_hdr (struct avf_fdir_conf *rcfg, int layer,
   struct virtchnl_proto_hdrs *hdrs;
 
   hdrs = &rcfg->add_fltr.rule_cfg.proto_hdrs;
-  if (layer > VIRTCHNL_MAX_NUM_PROTO_HDRS)
+  if (layer >= VIRTCHNL_MAX_NUM_PROTO_HDRS)
     return -1;
 
   hdrs->proto_hdr[layer].type = hdr;
@@ -486,7 +486,7 @@ int
 avf_fdir_rcfg_act_queue (struct avf_fdir_conf *rcfg, int queue, int size,
 			 int act_idx)
 {
-  if (act_idx > VIRTCHNL_MAX_NUM_ACTIONS)
+  if (act_idx >= VIRTCHNL_MAX_NUM_ACTIONS)
     return -AVF_FAILURE;
 
   struct virtchnl_filter_action *filter_action;
@@ -565,7 +565,7 @@ avf_fdir_rcfg_act_drop (struct avf_fdir_conf *rcfg, int act_idx)
 {
   struct virtchnl_filter_action *filter_action;
 
-  if (act_idx > VIRTCHNL_MAX_NUM_ACTIONS)
+  if (act_idx >= VIRTCHNL_MAX_NUM_ACTIONS)
     return -AVF_FAILURE;
 
   filter_action = rcfg->add_fltr.rule_cfg.action_set.actions + act_idx;
@@ -579,7 +579,7 @@ avf_fdir_rcfg_act_mark (struct avf_fdir_conf *rcfg, const u32 mark,
 			int act_idx)
 {
   struct virtchnl_filter_action *filter_action;
-  if (act_idx > VIRTCHNL_MAX_NUM_ACTIONS)
+  if (act_idx >= VIRTCHNL_MAX_NUM_ACTIONS)
     return -AVF_FAILURE;
 
   filter_action = rcfg->add_fltr.rule_cfg.action_set.actions + act_idx;