bridge: Prevent insertion of FDB entry with disallowed vlan
authorToshiaki Makita <makita.toshiaki@lab.ntt.co.jp>
Mon, 26 May 2014 06:15:53 +0000 (15:15 +0900)
committerDavid S. Miller <davem@davemloft.net>
Mon, 2 Jun 2014 20:38:23 +0000 (13:38 -0700)
commite0d7968ab6c8bce2437b36fa7f04117e333f196d
tree8b066b3ba7134569ba12b5dd21b31be2621adf3c
parentbfc5184b69cf9eeb286137640351c650c27f118a
bridge: Prevent insertion of FDB entry with disallowed vlan

br_handle_local_finish() is allowing us to insert an FDB entry with
disallowed vlan. For example, when port 1 and 2 are communicating in
vlan 10, and even if vlan 10 is disallowed on port 3, port 3 can
interfere with their communication by spoofed src mac address with
vlan id 10.

Note: Even if it is judged that a frame should not be learned, it should
not be dropped because it is destined for not forwarding layer but higher
layer. See IEEE 802.1Q-2011 8.13.10.

Signed-off-by: Toshiaki Makita <makita.toshiaki@lab.ntt.co.jp>
Acked-by: Vlad Yasevich <vyasevic@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_input.c
net/bridge/br_private.h
net/bridge/br_vlan.c