]> www.infradead.org Git - users/jedix/linux-maple.git/commit
netfilter: nf_tables: fix use-after-free when deleting compat expressions
authorFlorian Westphal <fw@strlen.de>
Mon, 12 Nov 2018 21:43:45 +0000 (22:43 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 17 Dec 2018 08:28:48 +0000 (09:28 +0100)
commit38e22947994de227bc1ac57ec82314c24a605605
tree5cdaccac07774e5a085551e5005e89b05c1ab699
parentcae5446b33f65162fe21fdb633a63619633e7ea1
netfilter: nf_tables: fix use-after-free when deleting compat expressions

[ Upstream commit 29e3880109e357fdc607b4393f8308cef6af9413 ]

nft_compat ops do not have static storage duration, unlike all other
expressions.

When nf_tables_expr_destroy() returns, expr->ops might have been
free'd already, so we need to store next address before calling
expression destructor.

For same reason, we can't deref match pointer after nft_xt_put().

This can be easily reproduced by adding msleep() before
nft_match_destroy() returns.

Fixes: 0ca743a55991 ("netfilter: nf_tables: add compatibility layer for x_tables")
Reported-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
net/netfilter/nf_tables_api.c
net/netfilter/nft_compat.c