diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2016-08-05 09:23:33 +0200 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2016-12-22 02:26:05 +0100 |
| commit | a878ba45729e28709f5916b0ff04bd1620751d60 (patch) | |
| tree | 9727a4d198b010b9b336d2f305ff0abc11d259b8 /pimd/pim_igmpv3.c | |
| parent | pimd: Send hello immediately with receive of new genid (diff) | |
| download | frr-a878ba45729e28709f5916b0ff04bd1620751d60.tar.xz frr-a878ba45729e28709f5916b0ff04bd1620751d60.zip | |
pimd: Some SA analysis fixes.
This commit addresses some SA issues found:
1) double set variables
2) value set but never used.
3) wrong return path used.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'pimd/pim_igmpv3.c')
| -rw-r--r-- | pimd/pim_igmpv3.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/pimd/pim_igmpv3.c b/pimd/pim_igmpv3.c index 79932edb3..149d18df5 100644 --- a/pimd/pim_igmpv3.c +++ b/pimd/pim_igmpv3.c @@ -1134,9 +1134,6 @@ static int group_retransmit_sources(struct igmp_group *group, struct igmp_source *src; int num_retransmit_sources_left = 0; - query_buf1_max_sources = (sizeof(query_buf1) - IGMP_V3_SOURCES_OFFSET) >> 2; - query_buf2_max_sources = (sizeof(query_buf2) - IGMP_V3_SOURCES_OFFSET) >> 2; - source_addr1 = (struct in_addr *)(query_buf1 + IGMP_V3_SOURCES_OFFSET); source_addr2 = (struct in_addr *)(query_buf2 + IGMP_V3_SOURCES_OFFSET); |