The "qv_info" ptr cannot be NULL when it gets the address of
an element of the flexible array "qvlist_info->qv_info".

Detected using the static analysis tool - Svace.

Signed-off-by: Alexey Kodanev <[email protected]>
---
 drivers/net/ethernet/intel/i40e/i40e_client.c      | 4 ----
 drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 4 ----
 2 files changed, 8 deletions(-)

diff --git a/drivers/net/ethernet/intel/i40e/i40e_client.c 
b/drivers/net/ethernet/intel/i40e/i40e_client.c
index 306758428aef..b32071ee84af 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_client.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_client.c
@@ -148,8 +148,6 @@ static void i40e_client_release_qvlist(struct i40e_info 
*ldev)
                u32 reg_idx;
 
                qv_info = &qvlist_info->qv_info[i];
-               if (!qv_info)
-                       continue;
                reg_idx = I40E_PFINT_LNKLSTN(qv_info->v_idx - 1);
                wr32(&pf->hw, reg_idx, I40E_PFINT_LNKLSTN_FIRSTQ_INDX_MASK);
        }
@@ -576,8 +574,6 @@ static int i40e_client_setup_qvlist(struct i40e_info *ldev,
 
        for (i = 0; i < qvlist_info->num_vectors; i++) {
                qv_info = &qvlist_info->qv_info[i];
-               if (!qv_info)
-                       continue;
                v_idx = qv_info->v_idx;
 
                /* Validate vector id belongs to this client */
diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c 
b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
index b34c71770887..83a34e98bdc7 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
@@ -491,8 +491,6 @@ static void i40e_release_rdma_qvlist(struct i40e_vf *vf)
                u32 v_idx, reg_idx, reg;
 
                qv_info = &qvlist_info->qv_info[i];
-               if (!qv_info)
-                       continue;
                v_idx = qv_info->v_idx;
                if (qv_info->ceq_idx != I40E_QUEUE_INVALID_IDX) {
                        /* Figure out the queue after CEQ and make that the
@@ -562,8 +560,6 @@ i40e_config_rdma_qvlist(struct i40e_vf *vf,
        msix_vf = pf->hw.func_caps.num_msix_vectors_vf;
        for (i = 0; i < qvlist_info->num_vectors; i++) {
                qv_info = &qvlist_info->qv_info[i];
-               if (!qv_info)
-                       continue;
 
                /* Validate vector id belongs to this vf */
                if (!i40e_vc_isvalid_vector_id(vf, qv_info->v_idx)) {
-- 
2.25.1

Reply via email to