]> git.baikalelectronics.ru Git - kernel.git/commit
netfilter: nf_tables: fix null deref due to zeroed list head
authorFlorian Westphal <fw@strlen.de>
Tue, 9 Aug 2022 16:34:02 +0000 (18:34 +0200)
committerPablo Neira Ayuso <pablo@netfilter.org>
Tue, 9 Aug 2022 18:13:30 +0000 (20:13 +0200)
commitf89eacf59f83e31df9db1b3a4463c93fe8fc6bce
treeb8c02009edeb09bf67fa12ab291b3e6dea4b55f2
parenta3a86ece7e35d697dc7871543889ce8eb8a83e52
netfilter: nf_tables: fix null deref due to zeroed list head

In nf_tables_updtable, if nf_tables_table_enable returns an error,
nft_trans_destroy is called to free the transaction object.

nft_trans_destroy() calls list_del(), but the transaction was never
placed on a list -- the list head is all zeroes, this results in
a null dereference:

BUG: KASAN: null-ptr-deref in nft_trans_destroy+0x26/0x59
Call Trace:
 nft_trans_destroy+0x26/0x59
 nf_tables_newtable+0x4bc/0x9bc
 [..]

Its sane to assume that nft_trans_destroy() can be called
on the transaction object returned by nft_trans_alloc(), so
make sure the list head is initialised.

Fixes: 24afa2cc4521 ("netfilter: nf_tables: use new transaction infrastructure to handle table")
Reported-by: mingi cho <mgcho.minic@gmail.com>
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nf_tables_api.c