]> git.baikalelectronics.ru Git - kernel.git/commit
net: bridge: delete local fdb on device init failure
authorNikolay Aleksandrov <nikolay@cumulusnetworks.com>
Mon, 29 Jul 2019 09:28:41 +0000 (12:28 +0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 29 Jul 2019 16:50:05 +0000 (09:50 -0700)
commit7ac0d732f0f8625c0082890a64ff5318459b5901
treede2cdab15d1c67e21c0e1ec79f5c0cd1349c0be7
parent26cb7943d4b3185489435b7e29cb8e9da0719611
net: bridge: delete local fdb on device init failure

On initialization failure we have to delete the local fdb which was
inserted due to the default pvid creation. This problem has been present
since the inception of default_pvid. Note that currently there are 2 cases:
1) in br_dev_init() when br_multicast_init() fails
2) if register_netdevice() fails after calling ndo_init()

This patch takes care of both since br_vlan_flush() is called on both
occasions. Also the new fdb delete would be a no-op on normal bridge
device destruction since the local fdb would've been already flushed by
br_dev_delete(). This is not an issue for ports since nbp_vlan_init() is
called last when adding a port thus nothing can fail after it.

Reported-by: syzbot+88533dc8b582309bf3ee@syzkaller.appspotmail.com
Fixes: e16c2cd1ee73 ("bridge: Add filtering support for default_pvid")
Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_vlan.c