[dpdk-dev] [PATCH v2 5/8] net/i40e/base: fix possible uninitialized variable

[Guinan Sun]

Fix possible uninitialized variable in i40e in the i40e_get_lpi_counters
function.

Fixes: 429bdc0cd967 ("net/i40e/base: add function to read LPI counters")
Cc: sta...@dpdk.org

Signed-off-by: Adam Ludkiewicz <adam.ludkiew...@intel.com>
Signed-off-by: Guinan Sun <guinanx....@intel.com>
---
V2:
* Modify commit message.
---
 drivers/net/i40e/base/i40e_common.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/i40e/base/i40e_common.c 
b/drivers/net/i40e/base/i40e_common.c
index 46a0b7881..85c22849e 100644
--- a/drivers/net/i40e/base/i40e_common.c
+++ b/drivers/net/i40e/base/i40e_common.c
@@ -7097,7 +7097,7 @@ enum i40e_status_code i40e_get_lpi_counters(struct 
i40e_hw *hw,
                                I40E_AQ_RUN_PHY_ACT_DNL_OPCODE_GET_EEE_STAT,
                                &cmd_status, tx_counter, rx_counter, NULL);
 
-               if (cmd_status != I40E_AQ_RUN_PHY_ACT_CMD_STAT_SUCC)
+               if (!retval && cmd_status != I40E_AQ_RUN_PHY_ACT_CMD_STAT_SUCC)
                        retval = I40E_ERR_ADMIN_QUEUE_ERROR;
 
                return retval;
-- 
2.17.1