Commit d1942cd4 authored by Ido Schimmel's avatar Ido Schimmel Committed by David S. Miller

bridge: mcast: Simplify MDB entry creation

Before creating a new MDB entry, br_multicast_new_group() will call
br_mdb_ip_get() to see if one exists and return it if so.

Therefore, simply call br_multicast_new_group() and omit the call to
br_mdb_ip_get().
Signed-off-by: default avatarIdo Schimmel <idosch@nvidia.com>
Acked-by: default avatarNikolay Aleksandrov <razor@blackwall.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 262985fa
...@@ -866,7 +866,6 @@ static int br_mdb_add_group(struct net_bridge *br, struct net_bridge_port *port, ...@@ -866,7 +866,6 @@ static int br_mdb_add_group(struct net_bridge *br, struct net_bridge_port *port,
unsigned long now = jiffies; unsigned long now = jiffies;
unsigned char flags = 0; unsigned char flags = 0;
u8 filter_mode; u8 filter_mode;
int err;
__mdb_entry_to_br_ip(entry, &group, mdb_attrs); __mdb_entry_to_br_ip(entry, &group, mdb_attrs);
...@@ -892,13 +891,9 @@ static int br_mdb_add_group(struct net_bridge *br, struct net_bridge_port *port, ...@@ -892,13 +891,9 @@ static int br_mdb_add_group(struct net_bridge *br, struct net_bridge_port *port,
return -EINVAL; return -EINVAL;
} }
mp = br_mdb_ip_get(br, &group); mp = br_multicast_new_group(br, &group);
if (!mp) { if (IS_ERR(mp))
mp = br_multicast_new_group(br, &group); return PTR_ERR(mp);
err = PTR_ERR_OR_ZERO(mp);
if (err)
return err;
}
/* host join */ /* host join */
if (!port) { if (!port) {
......
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