]> git.proxmox.com Git - mirror_ubuntu-focal-kernel.git/commit
bridge: mcast: Fix MLD2 Report IPv6 payload length check
authorLinus Lüssing <linus.luessing@c0d3.blue>
Sun, 5 Jul 2020 19:10:17 +0000 (21:10 +0200)
committerKhalid Elmously <khalid.elmously@canonical.com>
Sat, 8 Aug 2020 05:53:12 +0000 (01:53 -0400)
commit75a1b08f631de7f689197e42b24913deeb791b5a
tree72e183676f6011a6b664af1c34665e7e623c088f
parentff81b6d6a604254909e50ef17c1190ac7240b4b0
bridge: mcast: Fix MLD2 Report IPv6 payload length check

BugLink: https://bugs.launchpad.net/bugs/1888560
[ Upstream commit 5fc6266af7b427243da24f3443a50cd4584aac06 ]

Commit e57f61858b7c ("net: bridge: mcast: fix stale nsrcs pointer in
igmp3/mld2 report handling") introduced a bug in the IPv6 header payload
length check which would potentially lead to rejecting a valid MLD2 Report:

The check needs to take into account the 2 bytes for the "Number of
Sources" field in the "Multicast Address Record" before reading it.
And not the size of a pointer to this field.

Fixes: e57f61858b7c ("net: bridge: mcast: fix stale nsrcs pointer in igmp3/mld2 report handling")
Acked-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Kelsey Skunberg <kelsey.skunberg@canonical.com>
net/bridge/br_multicast.c