[PATCH v1 10/24] net/igc/base: fix iterator type

Anatoly Burakov anatoly.burakov at intel.com
Thu Feb 6 17:08:33 CET 2025


From: Amir Avivi <amir.avivi at intel.com>

Fix static analysis warning about comparison between types of
incompatible width, which might lead to an infinite loop due to overflow.

Fixes: 8cb7c57d9b3c ("net/igc: support device initialization")
Cc: stable at dpdk.org

Signed-off-by: Amir Avivi <amir.avivi at intel.com>
Signed-off-by: Anatoly Burakov <anatoly.burakov at intel.com>
---
 .mailmap                             | 1 +
 drivers/net/intel/igc/base/igc_phy.c | 3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/.mailmap b/.mailmap
index 3faff80f0e..23ec118ebc 100644
--- a/.mailmap
+++ b/.mailmap
@@ -84,6 +84,7 @@ Amaranath Somalapuram <asomalap at amd.com>
 Amine Kherbouche <amine.kherbouche at 6wind.com>
 Amin Tootoonchian <amint at icsi.berkeley.edu>
 Ami Sabo <amis at radware.com>
+Amir Avivi <amir.avivi at intel.com>
 Amit Bernstein <amitbern at amazon.com>
 Amit Gupta <agupta3 at marvell.com>
 Amit Prakash Shukla <amitprakashs at marvell.com>
diff --git a/drivers/net/intel/igc/base/igc_phy.c b/drivers/net/intel/igc/base/igc_phy.c
index 706f037f0e..dd56b994af 100644
--- a/drivers/net/intel/igc/base/igc_phy.c
+++ b/drivers/net/intel/igc/base/igc_phy.c
@@ -2366,7 +2366,8 @@ s32 igc_phy_has_link_generic(struct igc_hw *hw, u32 iterations,
 			       u32 usec_interval, bool *success)
 {
 	s32 ret_val = IGC_SUCCESS;
-	u16 i, phy_status;
+	u16 phy_status;
+	u32 i;
 
 	DEBUGFUNC("igc_phy_has_link_generic");
 
-- 
2.43.5



More information about the dev mailing list