summaryrefslogtreecommitdiff
path: root/net/l2tp
diff options
context:
space:
mode:
authorFlorian Westphal <fw@strlen.de>2018-03-09 16:27:31 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2018-04-08 15:26:29 +0300
commitbbfbc396f07680d5ca76a3a6e5e378e92f7d5ff7 (patch)
treee9feca2155b779f7e9e5e965de48e94d104a41a7 /net/l2tp
parent546ade704a98bfdcb7c6df76af14100a4ca53263 (diff)
downloadlinux-bbfbc396f07680d5ca76a3a6e5e378e92f7d5ff7.tar.xz
netfilter: bridge: ebt_among: add more missing match size checks
commit c8d70a700a5b486bfa8e5a7d33d805389f6e59f9 upstream. ebt_among is special, it has a dynamic match size and is exempt from the central size checks. commit c4585a2823edf ("bridge: ebt_among: add missing match size checks") added validation for pool size, but missed fact that the macros ebt_among_wh_src/dst can already return out-of-bound result because they do not check value of wh_src/dst_ofs (an offset) vs. the size of the match that userspace gave to us. v2: check that offset has correct alignment. Paolo Abeni points out that we should also check that src/dst wormhash arrays do not overlap, and src + length lines up with start of dst (or vice versa). v3: compact wormhash_sizes_valid() part NB: Fixes tag is intentionally wrong, this bug exists from day one when match was added for 2.6 kernel. Tag is there so stable maintainers will notice this one too. Tested with same rules from the earlier patch. Fixes: c4585a2823edf ("bridge: ebt_among: add missing match size checks") Reported-by: <syzbot+bdabab6f1983a03fc009@syzkaller.appspotmail.com> Signed-off-by: Florian Westphal <fw@strlen.de> Reviewed-by: Eric Dumazet <edumazet@google.com> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/l2tp')
0 files changed, 0 insertions, 0 deletions