]> git.baikalelectronics.ru Git - kernel.git/commit
netfilter: nf_tables: unbind set in rule from commit path
authorPablo Neira Ayuso <pablo@netfilter.org>
Sat, 2 Feb 2019 09:49:13 +0000 (10:49 +0100)
committerPablo Neira Ayuso <pablo@netfilter.org>
Mon, 4 Feb 2019 16:29:17 +0000 (17:29 +0100)
commitb80da8170b36bed0144fa27a05b5a559ea4a7374
tree87f5deb50b2c844809a6e00fcf44605461610e04
parent32fae5532fcbb80245c4fa904d9269c9ba1c6f9e
netfilter: nf_tables: unbind set in rule from commit path

Anonymous sets that are bound to rules from the same transaction trigger
a kernel splat from the abort path due to double set list removal and
double free.

This patch updates the logic to search for the transaction that is
responsible for creating the set and disable the set list removal and
release, given the rule is now responsible for this. Lookup is reverse
since the transaction that adds the set is likely to be at the tail of
the list.

Moreover, this patch adds the unbind step to deliver the event from the
commit path.  This should not be done from the worker thread, since we
have no guarantees of in-order delivery to the listener.

This patch removes the assumption that both activate and deactivate
callbacks need to be provided.

Fixes: ea3cf8101c0d ("netfilter: nf_tables: split set destruction in deactivate and destroy phase")
Reported-by: Mikhail Morfikov <mmorfikov@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
include/net/netfilter/nf_tables.h
net/netfilter/nf_tables_api.c
net/netfilter/nft_compat.c
net/netfilter/nft_dynset.c
net/netfilter/nft_immediate.c
net/netfilter/nft_lookup.c
net/netfilter/nft_objref.c