diff options
| author | Mobashshera Rasool <mrasool@vmware.com> | 2020-12-17 12:40:13 +0100 |
|---|---|---|
| committer | Mobashshera Rasool <mrasool@vmware.com> | 2020-12-24 07:39:12 +0100 |
| commit | d1b61cb92dfdb4815d5f729a8f89ceb6eaee7ba6 (patch) | |
| tree | 7634ca58c11551a909530386aa9ccf141634e6b0 /pimd/pim_igmp.c | |
| parent | Merge pull request #7744 from donaldsharp/end_configuration (diff) | |
| download | frr-d1b61cb92dfdb4815d5f729a8f89ceb6eaee7ba6.tar.xz frr-d1b61cb92dfdb4815d5f729a8f89ceb6eaee7ba6.zip | |
pimd: IGMP conformance 5.10 test case is failing
Test case 5.10 sends leave message to unicast address, the leave
packet is accepted and a query message is sent in response to this.
No validation for address is present in the function
Add check for addresses as per RFC. Leave messages are allowed only
sent to either ALL-ROUTERS (224.0.0.2) or group address.
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>
Diffstat (limited to 'pimd/pim_igmp.c')
| -rw-r--r-- | pimd/pim_igmp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pimd/pim_igmp.c b/pimd/pim_igmp.c index 9924e335b..73e42e9d8 100644 --- a/pimd/pim_igmp.c +++ b/pimd/pim_igmp.c @@ -558,8 +558,8 @@ int pim_igmp_packet(struct igmp_sock *igmp, char *buf, size_t len) igmp_msg, igmp_msg_len); case PIM_IGMP_V2_LEAVE_GROUP: - return igmp_v2_recv_leave(igmp, ip_hdr->ip_src, from_str, - igmp_msg, igmp_msg_len); + return igmp_v2_recv_leave(igmp, ip_hdr, from_str, igmp_msg, + igmp_msg_len); case PIM_IGMP_MTRACE_RESPONSE: return igmp_mtrace_recv_response(igmp, ip_hdr, ip_hdr->ip_src, |