diff options
| author | Pablo Neira Ayuso <pablo@netfilter.org> | 2014-05-10 18:46:02 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2014-05-12 16:33:10 +0200 |
| commit | f7e7e39b21c285ad73a62fac0736191b8d830704 (patch) | |
| tree | 8fbe1435d12728c7fadd09f8fc829437d365492d /net | |
| parent | 7b9d5ef932297413adcbd8be98fe612b9527a312 (diff) | |
| download | linux-f7e7e39b21c285ad73a62fac0736191b8d830704.tar.gz linux-f7e7e39b21c285ad73a62fac0736191b8d830704.tar.bz2 linux-f7e7e39b21c285ad73a62fac0736191b8d830704.zip | |
netfilter: nf_tables: fix bogus rulenum after goto action
After returning from the chain that we just went to with no matchings,
we get a bogus rule number in the trace. To fix this, we would need
to iterate over the list of remaining rules in the chain to update the
rule number counter.
Patrick suggested to set this to the maximum value since the default
base chain policy is the very last action when the processing the base
chain is over.
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net')
| -rw-r--r-- | net/netfilter/nf_tables_core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netfilter/nf_tables_core.c b/net/netfilter/nf_tables_core.c index f55fb28264fa..be08a96b4f45 100644 --- a/net/netfilter/nf_tables_core.c +++ b/net/netfilter/nf_tables_core.c @@ -202,7 +202,7 @@ next_rule: } if (unlikely(pkt->skb->nf_trace)) - nft_trace_packet(pkt, basechain, ++rulenum, NFT_TRACE_POLICY); + nft_trace_packet(pkt, basechain, -1, NFT_TRACE_POLICY); rcu_read_lock_bh(); stats = rcu_dereference(nft_base_chain(basechain)->stats); |