netfilter: nf_tables: drop map element references from preparation phase
authorPablo Neira Ayuso <pablo@netfilter.org>
Fri, 16 Jun 2023 12:51:49 +0000 (14:51 +0200)
committerPablo Neira Ayuso <pablo@netfilter.org>
Tue, 20 Jun 2023 20:43:40 +0000 (22:43 +0200)
commit628bd3e49cba1c066228e23d71a852c23e26da73
treefe157a8fb8f5f228cc956af0ee5cad9b0c2782e3
parent26b5a5712eb85e253724e56a54c17f8519bd8e4e
netfilter: nf_tables: drop map element references from preparation phase

set .destroy callback releases the references to other objects in maps.
This is very late and it results in spurious EBUSY errors. Drop refcount
from the preparation phase instead, update set backend not to drop
reference counter from set .destroy path.

Exceptions: NFT_TRANS_PREPARE_ERROR does not require to drop the
reference counter because the transaction abort path releases the map
references for each element since the set is unbound. The abort path
also deals with releasing reference counter for new elements added to
unbound sets.

Fixes: 591054469b3e ("netfilter: nf_tables: revisit chain/object refcounting from elements")
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
include/net/netfilter/nf_tables.h
net/netfilter/nf_tables_api.c
net/netfilter/nft_set_bitmap.c
net/netfilter/nft_set_hash.c
net/netfilter/nft_set_pipapo.c
net/netfilter/nft_set_rbtree.c