netfilter: ebtables: reject non-bridge targets
authorFlorian Westphal <fw@strlen.de>
Wed, 6 Jun 2018 10:14:56 +0000 (12:14 +0200)
committerPablo Neira Ayuso <pablo@netfilter.org>
Wed, 6 Jun 2018 13:04:04 +0000 (15:04 +0200)
commit11ff7288beb2b7da889a014aff0a7b80bf8efcf3
treed0d38231a43b4853981a7f198c934d52dfde9d5f
parent9e8c8dabb78e886ace989729e763d28c76f5169e
netfilter: ebtables: reject non-bridge targets

the ebtables evaluation loop expects targets to return
positive values (jumps), or negative values (absolute verdicts).

This is completely different from what xtables does.
In xtables, targets are expected to return the standard netfilter
verdicts, i.e. NF_DROP, NF_ACCEPT, etc.

ebtables will consider these as jumps.

Therefore reject any target found due to unspec fallback.
v2: also reject watchers.  ebtables ignores their return value, so
a target that assumes skb ownership (and returns NF_STOLEN) causes
use-after-free.

The only watchers in the 'ebtables' front-end are log and nflog;
both have AF_BRIDGE specific wrappers on kernel side.

Reported-by: syzbot+2b43f681169a2a0d306a@syzkaller.appspotmail.com
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/bridge/netfilter/ebtables.c