On Sun, Feb 17, 2019 at 03:29:22PM +0100, Sebastian Gottschall wrote:
according to user reports this patch will cause a serious regression. igmpIs this also a problem in 4.20? This patch went into 4.20-rc1, so it
snooping is not working anymore with this patch
Am 02.11.2018 um 19:34 schrieb Greg Kroah-Hartman:
4.19-stable review patch. If anyone has any objections, please let me know.
------------------
From: Hangbin Liu <liuhangbin@xxxxxxxxx>
[ Upstream commit 5a2de63fd1a59c30c02526d427bc014b98adf508 ]
Based on RFC 4541, 2.1.1. IGMP Forwarding Rules
ÂÂÂ The switch supporting IGMP snooping must maintain a list of
 multicast routers and the ports on which they are attached. This
ÂÂÂ list can be constructed in any combination of the following ways:
ÂÂÂ a) This list should be built by the snooping switch sending
ÂÂÂÂÂÂ Multicast Router Solicitation messages as described in IGMP
 Multicast Router Discovery [MRDISC]. It may also snoop
ÂÂÂÂÂÂ Multicast Router Advertisement messages sent by and to other
ÂÂÂÂÂÂ nodes.
ÂÂÂ b) The arrival port for IGMP Queries (sent by multicast routers)
ÂÂÂÂÂÂ where the source address is not 0.0.0.0.
We should not add the port to router list when receives query with source
0.0.0.0.
Reported-by: Ying Xu <yinxu@xxxxxxxxxx>
Signed-off-by: Hangbin Liu <liuhangbin@xxxxxxxxx>
Acked-by: Nikolay Aleksandrov <nikolay@xxxxxxxxxxxxxxxxxxx>
Acked-by: Roopa Prabhu <roopa@xxxxxxxxxxxxxxxxxxx>
Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
ÂÂ net/bridge/br_multicast.c |ÂÂ 10 +++++++++-
ÂÂ 1 file changed, 9 insertions(+), 1 deletion(-)
--- a/net/bridge/br_multicast.c
+++ b/net/bridge/br_multicast.c
@@ -1420,7 +1420,15 @@ static void br_multicast_query_received(
ÂÂÂÂÂÂÂÂÂÂ return;
ÂÂÂÂÂÂ br_multicast_update_query_timer(br, query, max_delay);
-ÂÂÂ br_multicast_mark_router(br, port);
+
+ÂÂÂ /* Based on RFC4541, section 2.1.1 IGMP Forwarding Rules,
+ÂÂÂÂ * the arrival port for IGMP Queries where the source address
+ÂÂÂÂ * is 0.0.0.0 should not be added to router port list.
+ÂÂÂÂ */
+ÂÂÂ if ((saddr->proto == htons(ETH_P_IP) && saddr->u.ip4) ||
+ÂÂÂÂÂÂÂ (saddr->proto == htons(ETH_P_IPV6) &&
+ÂÂÂÂÂÂÂÂ !ipv6_addr_any(&saddr->u.ip6)))
+ÂÂÂÂÂÂÂ br_multicast_mark_router(br, port);
ÂÂ }
ÂÂ static void br_ip4_multicast_query(struct net_bridge *br,
has been around for a while with no reported issues that I can find.
Any pointers to the reports?
thanks,
greg k-h