avf: fix coverity issues 92/31792/3
authorChenmin Sun <chenmin.sun@intel.com>
Mon, 29 Mar 2021 19:50:02 +0000 (03:50 +0800)
committerDamjan Marion <dmarion@me.com>
Tue, 30 Mar 2021 09:40:03 +0000 (09:40 +0000)
CID #220403 ~ 220406

Type: fix

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

src/plugins/avf/avf_fdir_lib.c

index 44c10dc..f38614e 100644 (file)
@@ -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;