bridge: netfilter: orphan skb before invoking ip netfilter hooks
authorFlorian Westphal <fw@strlen.de>
Thu, 24 Oct 2013 19:32:42 +0000 (21:32 +0200)
committerPablo Neira Ayuso <pablo@netfilter.org>
Sun, 27 Oct 2013 20:44:33 +0000 (21:44 +0100)
commit6b8dbcf2c44fd7aa716560d04e9857c870bd510c
tree6a27588d35da7f83f10a4a299cdb98062570dc9f
parent1e56555ee1f3d04d89bdaa3df57ad2da9fbc0999
bridge: netfilter: orphan skb before invoking ip netfilter hooks

Pekka Pietikäinen reports xt_socket behavioural change after commit
00028aa37098o (netfilter: xt_socket: use IP early demux).

Reason is xt_socket now no longer does an unconditional sk lookup -
it re-uses existing skb->sk if possible, assuming ->sk was set by
ip early demux.

However, when netfilter is invoked via bridge, this can cause 'bogus'
sockets to be examined by the match, e.g. a 'tun' device socket.

bridge netfilter should orphan the skb just like the routing path
before invoking ipv4/ipv6 netfilter hooks to avoid this.

Reported-and-tested-by: Pekka Pietikäinen <pp@ee.oulu.fi>
Signed-off-by: Florian Westphal <fw@strlen.de>
Acked-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/bridge/br_netfilter.c