Commit 07dc8bc9 authored by Colin Ian King's avatar Colin Ian King Committed by Pablo Neira Ayuso
Browse files

netfilter: remove redundant assignment to e 



The assignment to variable e is redundant since the same assignment
occurs just a few lines later, hence it can be removed.  Cleans up
clang warning for arp_tables, ip_tables and ip6_tables:

warning: Value stored to 'e' is never read

Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarPablo Neira Ayuso <pablo@netfilter.org>
parent 32a72bbd

net/ipv4/netfilter/arp_tables.c

+0 −1
Original line number Diff line number Diff line
@@ -373,7 +373,6 @@ static int mark_source_chains(const struct xt_table_info *newinfo, 
					if (!xt_find_jump_offset(offsets, newpos,

								 newinfo->number))

						return 0;

					e = entry0 + newpos;

				} else {

					/* ... this is a fallthru */

					newpos = pos + e->next_offset;

net/ipv4/netfilter/ip_tables.c

+0 −1
Original line number Diff line number Diff line
@@ -439,7 +439,6 @@ mark_source_chains(const struct xt_table_info *newinfo, 
					if (!xt_find_jump_offset(offsets, newpos,

								 newinfo->number))

						return 0;

					e = entry0 + newpos;

				} else {

					/* ... this is a fallthru */

					newpos = pos + e->next_offset;

net/ipv6/netfilter/ip6_tables.c

+0 −1
Original line number Diff line number Diff line
@@ -458,7 +458,6 @@ mark_source_chains(const struct xt_table_info *newinfo, 
					if (!xt_find_jump_offset(offsets, newpos,

								 newinfo->number))

						return 0;

					e = entry0 + newpos;

				} else {

					/* ... this is a fallthru */

					newpos = pos + e->next_offset;

CRA Git | Maintained and supported by SUSTech CRA and CCSE