Commit 8fff1722 authored Jun 14, 2016 by Liping Zhang Committed by Pablo Neira Ayuso Jun 15, 2016

netfilter: nf_tables: fix a wrong check to skip the inactive rules



nft_genmask_cur has already done left-shift operator on the gencursor,
so there's no need to do left-shift operator on it again.

Fixes: ea4bd995 ("netfilter: nf_tables: add transaction helper functions")
Cc: Patrick McHardy <kaber@trash.net>
Signed-off-by: Liping Zhang <liping.zhang@spreadtrum.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>

parent a02f4248

net/netfilter/nf_tables_core.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -143,7 +143,7 @@ nft_do_chain(struct nft_pktinfo pkt, void priv)
		list_for_each_entry_continue_rcu(rule, &chain->rules, list) {

		/* This rule is not active, skip. */
		if (unlikely(rule->genmask & (1 << gencursor)))
		if (unlikely(rule->genmask & gencursor))
		continue;

		rulenum++;