diff options
| author | Pablo Neira Ayuso <pablo@netfilter.org> | 2023-07-20 09:17:21 +0200 |
|---|---|---|
| committer | Florian Westphal <fw@strlen.de> | 2023-07-20 17:21:11 +0200 |
| commit | 6eaf41e87a223ae6f8e7a28d6e78384ad7e407f8 (patch) | |
| tree | 6d1486294a30132bb59145127a97a7536f4f612b /net | |
| parent | netfilter: nf_tables: skip bound chain in netns release path (diff) | |
| download | linux-6eaf41e87a223ae6f8e7a28d6e78384ad7e407f8.tar.xz linux-6eaf41e87a223ae6f8e7a28d6e78384ad7e407f8.zip | |
netfilter: nf_tables: skip bound chain on rule flush
Skip bound chain when flushing table rules, the rule that owns this
chain releases these objects.
Otherwise, the following warning is triggered:
WARNING: CPU: 2 PID: 1217 at net/netfilter/nf_tables_api.c:2013 nf_tables_chain_destroy+0x1f7/0x210 [nf_tables]
CPU: 2 PID: 1217 Comm: chain-flush Not tainted 6.1.39 #1
RIP: 0010:nf_tables_chain_destroy+0x1f7/0x210 [nf_tables]
Fixes: d0e2c7de92c7 ("netfilter: nf_tables: add NFT_CHAIN_BINDING")
Reported-by: Kevin Rich <kevinrich1337@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: Florian Westphal <fw@strlen.de>
Diffstat (limited to 'net')
| -rw-r--r-- | net/netfilter/nf_tables_api.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/netfilter/nf_tables_api.c b/net/netfilter/nf_tables_api.c index 40bff6e2ea5d..b9a4d3fd1d34 100644 --- a/net/netfilter/nf_tables_api.c +++ b/net/netfilter/nf_tables_api.c @@ -4087,6 +4087,8 @@ static int nf_tables_delrule(struct sk_buff *skb, const struct nfnl_info *info, list_for_each_entry(chain, &table->chains, list) { if (!nft_is_active_next(net, chain)) continue; + if (nft_chain_is_bound(chain)) + continue; ctx.chain = chain; err = nft_delrule_by_chain(&ctx); |