eth: mlx5: avoid iterator use outside of a loop
authorJakub Kicinski <kuba@kernel.org>
Thu, 20 Apr 2023 01:58:02 +0000 (18:58 -0700)
committerJakub Kicinski <kuba@kernel.org>
Fri, 21 Apr 2023 02:25:05 +0000 (19:25 -0700)
Fix the following warning about risky iterator use:

drivers/net/ethernet/mellanox/mlx5/core/eq.c:1010 mlx5_comp_irq_get_affinity_mask() warn: iterator used outside loop: 'eq'

Acked-by: Saeed Mahameed <saeed@kernel.org>
Link: https://lore.kernel.org/r/20230420015802.815362-1-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/mellanox/mlx5/core/eq.c

index eb41f0a..1c35d72 100644 (file)
@@ -1070,10 +1070,11 @@ mlx5_comp_irq_get_affinity_mask(struct mlx5_core_dev *dev, int vector)
 
        list_for_each_entry(eq, &table->comp_eqs_list, list) {
                if (i++ == vector)
-                       break;
+                       return mlx5_irq_get_affinity_mask(eq->core.irq);
        }
 
-       return mlx5_irq_get_affinity_mask(eq->core.irq);
+       WARN_ON_ONCE(1);
+       return NULL;
 }
 EXPORT_SYMBOL(mlx5_comp_irq_get_affinity_mask);