From 21064cec96b833c78505022fc29f25bed9f2fe9a Mon Sep 17 00:00:00 2001 From: Neale Ranns Date: Wed, 26 Sep 2018 06:51:39 -0700 Subject: IGMP: handle (*,G) report with no source addresses Change-Id: I363370b9d4a27b992bad55c48fc930a2fbea2165 Signed-off-by: Neale Ranns --- src/plugins/igmp/igmp_report.c | 22 ++++++++++++++-------- test/test_igmp.py | 16 ++++++++++++++++ 2 files changed, 30 insertions(+), 8 deletions(-) diff --git a/src/plugins/igmp/igmp_report.c b/src/plugins/igmp/igmp_report.c index 2caa9361b70..fc8155d59e1 100644 --- a/src/plugins/igmp/igmp_report.c +++ b/src/plugins/igmp/igmp_report.c @@ -32,15 +32,21 @@ igmp_group_mk_source_list (const igmp_membership_group_v3_t * r) n = clib_net_to_host_u16 (r->n_src_addresses); if (0 == n) - return (NULL); - - vec_validate (srcs, n - 1); - s = r->src_addresses; - - for (ii = 0; ii < n; ii++) { - srcs[ii].ip4 = *s; - s++; + /* a (*,G) join has no source address specified */ + vec_validate (srcs, 0); + srcs[0].ip4.as_u32 = 0; + } + else + { + vec_validate (srcs, n - 1); + s = r->src_addresses; + + for (ii = 0; ii < n; ii++) + { + srcs[ii].ip4 = *s; + s++; + } } return (srcs); diff --git a/test/test_igmp.py b/test/test_igmp.py index cfdd1a8aed3..128ac5e8eff 100644 --- a/test/test_igmp.py +++ b/test/test_igmp.py @@ -628,6 +628,22 @@ class TestIgmp(VppTestCase): self.assertFalse(self.vapi.igmp_dump()) self.assertFalse(self.vapi.igmp_dump()) + # + # A (*,G) host report + # + p_j = (Ether(dst=self.pg0.local_mac, src=self.pg0.remote_mac) / + IP(src=self.pg0.remote_ip4, dst="224.0.0.22", tos=0xc0, ttl=1, + options=[IPOption(copy_flag=1, optclass="control", + option="router_alert")]) / + IGMPv3(type="Version 3 Membership Report") / + IGMPv3mr(numgrp=1) / + IGMPv3gr(rtype="Allow New Sources", maddr="239.1.1.2")) + + self.send(self.pg0, p_j) + + self.assertTrue(wait_for_igmp_event(self, 1, self.pg0, + "239.1.1.2", "0.0.0.0", 1)) + # # disable router config # -- cgit 1.2.3-korg