Identified by LGTM, the loop was incorrectly incrementing a different
variable and conditional on another.

Fixes: fe2b986ac662 ("net/dpaa2: support generic flow")
Cc: sunil.k...@nxp.com

Suggested-by: Ferruh Yigit <ferruh.yi...@intel.com>
Signed-off-by: Shreyansh Jain <shreyansh.j...@nxp.com>
---
 drivers/net/dpaa2/dpaa2_flow.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/dpaa2/dpaa2_flow.c b/drivers/net/dpaa2/dpaa2_flow.c
index 98f68139b..9ef46d47e 100644
--- a/drivers/net/dpaa2/dpaa2_flow.c
+++ b/drivers/net/dpaa2/dpaa2_flow.c
@@ -1734,7 +1734,7 @@ dpaa2_dev_verify_patterns(struct dpaa2_dev_priv *dev_priv,
        /* more than DPKG_MAX_NUM_OF_EXTRACTS. Verify this limitation too. */
        for (i = 0; pattern[i].type != RTE_FLOW_ITEM_TYPE_END; i++) {
                for (j = 0; j < MAX_TCS + 1; j++) {
-                               for (k = 0; k < DPKG_MAX_NUM_OF_EXTRACTS; j++) {
+                               for (k = 0; k < DPKG_MAX_NUM_OF_EXTRACTS; k++) {
                                        if 
(dev_priv->pattern[j].pattern_type[k] == pattern[i].type)
                                                break;
                                }
-- 
2.17.1

Reply via email to