Commit 59bb1d69 authored by Gavrilov Ilia's avatar Gavrilov Ilia Committed by Paolo Abeni
Browse files

ipv6: mcast: Remove redundant comparison in igmp6_mcf_get_next() 



The 'state->im' value will always be non-zero after
the 'while' statement, so the check can be removed.

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

Fixes: 1da177e4 ("Linux-2.6.12-rc2")
Signed-off-by: default avatarGavrilov Ilia <Ilia.Gavrilov@infotecs.ru>
Reviewed-by: default avatarDavid Ahern <dsahern@kernel.org>
Link: https://lore.kernel.org/r/20230912084100.1502379-1-Ilia.Gavrilov@infotecs.ru


Signed-off-by: default avatarPaolo Abeni <pabeni@redhat.com>
parent a613ed1a

net/ipv6/mcast.c

+0 −2
Original line number Diff line number Diff line
@@ -3011,8 +3011,6 @@ static struct ip6_sf_list *igmp6_mcf_get_next(struct seq_file *seq, struct ip6_s 
				continue;

			state->im = rcu_dereference(state->idev->mc_list);

		}

		if (!state->im)

			break;

		psf = rcu_dereference(state->im->mca_sources);

	}

out: