[PATCH] net/iavf: remove redundant assignment in FDIR parser

[PATCH] net/iavf: remove redundant assignment in FDIR parser

[Evgeny Sokolov]

From: Sokolov Evgeny <Evgeny.Sokolov@infotecs.ru>

The assignment to next_type in the <case> branch is redundant since
the value is never used afterwards.

Remove the leftover assignment, which appears to have been copied from
another parser branch, to avoid confusing readers.

Found by InfoTeCS on behalf of Linux Verification Center (linuxtesting.org) with SVACE.

Signed-off-by: Sokolov Evgeny <Evgeny.Sokolov@infotecs.ru>
---
 drivers/net/intel/iavf/iavf_fdir.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/net/intel/iavf/iavf_fdir.c b/drivers/net/intel/iavf/iavf_fdir.c
index ea620863ca..a5fe90283f 100644
--- a/drivers/net/intel/iavf/iavf_fdir.c
+++ b/drivers/net/intel/iavf/iavf_fdir.c
@@ -711,7 +711,6 @@ iavf_fdir_parse_pattern(__rte_unused struct iavf_adapter *ad,
 			ipv4_spec = item->spec;
 			ipv4_last = item->last;
 			ipv4_mask = item->mask;
-			next_type = (item + 1)->type;
 
 			hdr = &hdrs->proto_hdr[layer];
 
-- 
2.30.2

Re: [PATCH] net/iavf: remove redundant assignment in FDIR parser

[Burakov, Anatoly]

On 9/15/2026 11:50 AM, Evgeny Sokolov wrote:
> From: Sokolov Evgeny <Evgeny.Sokolov@infotecs.ru>
> 
> The assignment to next_type in the <case> branch is redundant since
> the value is never used afterwards.
> 
> Remove the leftover assignment, which appears to have been copied from
> another parser branch, to avoid confusing readers.
> 
> Found by InfoTeCS on behalf of Linux Verification Center (linuxtesting.org) with SVACE.
> 
> Signed-off-by: Sokolov Evgeny <Evgeny.Sokolov@infotecs.ru>
> ---

This patch looks related to the other one [1] that you have submitted, 
and my comments for that patch apply here as well.

Also, please do not put advertisements in commit message, thank you.

[1] 
https://patches.dpdk.org/project/dpdk/patch/20260915104617.467149-1-Evgeny.Sokolov@infotecs.ru/

-- 
Thanks,
Anatoly