Currently, one of the Tx SA table accesses uses `i` iterator instead of the
`sa_index` variable that was assigned from said iterator. While this is
technically not a bug because `i` and `sa_index` always carry the same
value, it should still be fixed for readability purposes.

Signed-off-by: Anatoly Burakov <[email protected]>
---
 drivers/net/intel/ixgbe/ixgbe_ipsec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/intel/ixgbe/ixgbe_ipsec.c 
b/drivers/net/intel/ixgbe/ixgbe_ipsec.c
index 9c4937fcba..f026a41690 100644
--- a/drivers/net/intel/ixgbe/ixgbe_ipsec.c
+++ b/drivers/net/intel/ixgbe/ixgbe_ipsec.c
@@ -229,7 +229,7 @@ ixgbe_crypto_add_sa(struct ixgbe_crypto_session *ic_session)
 
                priv->tx_sa_tbl[sa_index].spi =
                        rte_cpu_to_be_32(ic_session->spi);
-               priv->tx_sa_tbl[i].used = 1;
+               priv->tx_sa_tbl[sa_index].used = 1;
                ic_session->sa_index = sa_index;
 
                memcpy(key, ic_session->key, ic_session->key_len);
-- 
2.52.0

Reply via email to