Commit b2bc58d4 authored by Vladimir Oltean's avatar Vladimir Oltean Committed by David S. Miller

net: bridge: vlan: check early for lack of BRENTRY flag in br_vlan_add_existing

When a VLAN is added to a bridge port, a master VLAN gets created on the
bridge for context, but it doesn't have the BRENTRY flag.

Then, when the same VLAN is added to the bridge itself, that enters
through the br_vlan_add_existing() code path and gains the BRENTRY flag,
thus it becomes "existing".

It seems natural to check for this condition early, because the current
code flow is to notify switchdev of the addition of a VLAN that isn't a
brentry, just to delete it immediately afterwards.
Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Acked-by: default avatarNikolay Aleksandrov <nikolay@nvidia.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent b0471c26
...@@ -679,16 +679,15 @@ static int br_vlan_add_existing(struct net_bridge *br, ...@@ -679,16 +679,15 @@ static int br_vlan_add_existing(struct net_bridge *br,
{ {
int err; int err;
/* Trying to change flags of non-existent bridge vlan */
if (!br_vlan_is_brentry(vlan) && !(flags & BRIDGE_VLAN_INFO_BRENTRY))
return -EINVAL;
err = br_switchdev_port_vlan_add(br->dev, vlan->vid, flags, extack); err = br_switchdev_port_vlan_add(br->dev, vlan->vid, flags, extack);
if (err && err != -EOPNOTSUPP) if (err && err != -EOPNOTSUPP)
return err; return err;
if (!br_vlan_is_brentry(vlan)) { if (!br_vlan_is_brentry(vlan)) {
/* Trying to change flags of non-existent bridge vlan */
if (!(flags & BRIDGE_VLAN_INFO_BRENTRY)) {
err = -EINVAL;
goto err_flags;
}
/* It was only kept for port vlans, now make it real */ /* It was only kept for port vlans, now make it real */
err = br_fdb_add_local(br, NULL, br->dev->dev_addr, vlan->vid); err = br_fdb_add_local(br, NULL, br->dev->dev_addr, vlan->vid);
if (err) { if (err) {
...@@ -709,7 +708,6 @@ static int br_vlan_add_existing(struct net_bridge *br, ...@@ -709,7 +708,6 @@ static int br_vlan_add_existing(struct net_bridge *br,
return 0; return 0;
err_fdb_insert: err_fdb_insert:
err_flags:
br_switchdev_port_vlan_del(br->dev, vlan->vid); br_switchdev_port_vlan_del(br->dev, vlan->vid);
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment