net/bnxt: fix mismatched type comparison in MAC restore
authorAjit Khaparde <ajit.khaparde@broadcom.com>
Fri, 30 Apr 2021 20:14:10 +0000 (13:14 -0700)
committerAjit Khaparde <ajit.khaparde@broadcom.com>
Sun, 2 May 2021 22:17:17 +0000 (00:17 +0200)
dev_info.max_mac_addrs is of type uint32_t. But the counter i is
of type uint16_t. This mismatch may cause the loop condition may
always be true. Change the loop counter variable to uint32_t.

Fixes: b02f1573cd07 ("net/bnxt: restore MAC filters during reset recovery")
Cc: stable@dpdk.org
Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
drivers/net/bnxt/bnxt_ethdev.c

index 9c3e0f9..115acda 100644 (file)
@@ -3989,7 +3989,7 @@ static int bnxt_restore_mac_filters(struct bnxt *bp)
        struct rte_ether_addr *addr;
        uint64_t pool_mask;
        uint32_t pool = 0;
-       uint16_t i;
+       uint32_t i;
        int rc;
 
        if (BNXT_VF(bp) && !BNXT_VF_IS_TRUSTED(bp))