Skip to content

Commit

Permalink
net/bnxt: fix mismatched type comparison in MAC restore
Browse files Browse the repository at this point in the history
[ upstream commit c8e8154 ]

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: b02f157 ("net/bnxt: restore MAC filters during reset recovery")

Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
  • Loading branch information
ajitkhaparde authored and steevenlee committed Jun 8, 2021
1 parent 1cfcacc commit 91d4a17
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/net/bnxt/bnxt_ethdev.c
Expand Up @@ -3972,7 +3972,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))
Expand Down

0 comments on commit 91d4a17

Please sign in to comment.