]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
netfilter: nft_inner: incorrect percpu area handling under softirq
authorPablo Neira Ayuso <pablo@netfilter.org>
Wed, 27 Nov 2024 11:46:54 +0000 (12:46 +0100)
committerPablo Neira Ayuso <pablo@netfilter.org>
Tue, 3 Dec 2024 21:10:58 +0000 (22:10 +0100)
commit7b1d83da254be3bf054965c8f3b1ad976f460ae5
tree2bbb595d8e5a0aa12eae64747e813f8cd1b024bb
parentb7529880cb961d515642ce63f9d7570869bbbdc3
netfilter: nft_inner: incorrect percpu area handling under softirq

Softirq can interrupt ongoing packet from process context that is
walking over the percpu area that contains inner header offsets.

Disable bh and perform three checks before restoring the percpu inner
header offsets to validate that the percpu area is valid for this
skbuff:

1) If the NFT_PKTINFO_INNER_FULL flag is set on, then this skbuff
   has already been parsed before for inner header fetching to
   register.

2) Validate that the percpu area refers to this skbuff using the
   skbuff pointer as a cookie. If there is a cookie mismatch, then
   this skbuff needs to be parsed again.

3) Finally, validate if the percpu area refers to this tunnel type.

Only after these three checks the percpu area is restored to a on-stack
copy and bh is enabled again.

After inner header fetching, the on-stack copy is stored back to the
percpu area.

Fixes: 3a07327d10a0 ("netfilter: nft_inner: support for inner tunnel header matching")
Reported-by: syzbot+84d0441b9860f0d63285@syzkaller.appspotmail.com
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
include/net/netfilter/nf_tables_core.h
net/netfilter/nft_inner.c