Nikolay Aleksandrov
2020-Sep-15 14:57 UTC
[Bridge] [PATCH net-next] net: bridge: mcast: don't ignore return value of __grp_src_toex_excl
When we're handling TO_EXCLUDE report in EXCLUDE filter mode we should not ignore the return value of __grp_src_toex_excl() as we'll miss sending notifications about group changes. Fixes: 5bf1e00b6849 ("net: bridge: mcast: support for IGMPV3/MLDv2 CHANGE_TO_INCLUDE/EXCLUDE report") Signed-off-by: Nikolay Aleksandrov <nikolay at cumulusnetworks.com> --- net/bridge/br_multicast.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/bridge/br_multicast.c b/net/bridge/br_multicast.c index 33adf44ef7ec..e77f1e27caf7 100644 --- a/net/bridge/br_multicast.c +++ b/net/bridge/br_multicast.c @@ -1710,7 +1710,7 @@ static bool br_multicast_toex(struct net_bridge_port_group *pg, changed = true; break; case MCAST_EXCLUDE: - __grp_src_toex_excl(pg, srcs, nsrcs, src_size); + changed = __grp_src_toex_excl(pg, srcs, nsrcs, src_size); break; } -- 2.25.4
David Miller
2020-Sep-17 00:13 UTC
[Bridge] [PATCH net-next] net: bridge: mcast: don't ignore return value of __grp_src_toex_excl
From: Nikolay Aleksandrov <nikolay at cumulusnetworks.com> Date: Tue, 15 Sep 2020 17:57:24 +0300> When we're handling TO_EXCLUDE report in EXCLUDE filter mode we should > not ignore the return value of __grp_src_toex_excl() as we'll miss > sending notifications about group changes. > > Fixes: 5bf1e00b6849 ("net: bridge: mcast: support for IGMPV3/MLDv2 CHANGE_TO_INCLUDE/EXCLUDE report") > Signed-off-by: Nikolay Aleksandrov <nikolay at cumulusnetworks.com>Applied, thank you.