netfilter: nf_tables: release mutex after nft_gc_seq_end from abort path
authorPablo Neira Ayuso <pablo@netfilter.org>
Thu, 28 Mar 2024 13:23:55 +0000 (14:23 +0100)
committerPablo Neira Ayuso <pablo@netfilter.org>
Thu, 4 Apr 2024 09:34:42 +0000 (11:34 +0200)
commit0d459e2ffb541841714839e8228b845458ed3b27
treef8e8e2753b4d585c6db4d0a1603fe97f1fa11ae0
parenta45e6889575c2067d3c0212b6bc1022891e65b91
netfilter: nf_tables: release mutex after nft_gc_seq_end from abort path

The commit mutex should not be released during the critical section
between nft_gc_seq_begin() and nft_gc_seq_end(), otherwise, async GC
worker could collect expired objects and get the released commit lock
within the same GC sequence.

nf_tables_module_autoload() temporarily releases the mutex to load
module dependencies, then it goes back to replay the transaction again.
Move it at the end of the abort phase after nft_gc_seq_end() is called.

Cc: stable@vger.kernel.org
Fixes: 720344340fb9 ("netfilter: nf_tables: GC transaction race with abort path")
Reported-by: Kuan-Ting Chen <hexrabbit@devco.re>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nf_tables_api.c