app/testpmd: remove redundant policy action condition

[ upstream commit 4c2e746 ] The loop over policy actions will always exit when it sees the flow end action, so the next check is redundant. Link: https://pvs-studio.com/en/blog/posts/cpp/1179/ Fixes: f29fa2c ("app/testpmd: support policy actions per color") Signed-off-by: Stephen Hemminger <[email protected]> Acked-by: Bruce Richardson <[email protected]> Acked-by: Ajit Khaparde <[email protected]> Acked-by: Chengwen Feng <[email protected]>
DPDK · Dec 6, 2024 · 1fb0641 · 1fb0641
1 parent e5eb4ae
commit 1fb0641
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
@@ -2219,7 +2219,7 @@ port_meter_policy_add(portid_t port_id, uint32_t policy_id,
 		for (act_n = 0, start = act;
 			act->type != RTE_FLOW_ACTION_TYPE_END; act++)
 			act_n++;
-		if (act_n && act->type == RTE_FLOW_ACTION_TYPE_END)
+		if (act_n > 0)
 			policy.actions[i] = start;
 		else
 			policy.actions[i] = NULL;