diff options
author | David Lamparter <equinox@opensourcerouting.org> | 2021-06-25 10:53:26 +0200 |
---|---|---|
committer | David Lamparter <equinox@opensourcerouting.org> | 2021-11-17 11:17:44 +0100 |
commit | 4efdb9c6282485e48ffd387e10b0a0e116de0bb9 (patch) | |
tree | c3c3e250a1f9c4365913a5e7e545b6ae8da463cb /pimd/pim_rpf.c | |
parent | pimd: actually return msec in timer_remain_msec() (diff) | |
download | frr-4efdb9c6282485e48ffd387e10b0a0e116de0bb9.tar.xz frr-4efdb9c6282485e48ffd387e10b0a0e116de0bb9.zip |
pimd: clean up BSR NHT & fix parallel links
The Bootstrap message RX path needs a RPF check for the BSR address,
and this is implemented both incorrectly as well as quite ugly.
Clean up and fix case when we have multiple interfaces to the same LAN
and/or ECMP nexthops (both would cause message duplication, the former
can even cause BSM forwarding loops.)
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'pimd/pim_rpf.c')
-rw-r--r-- | pimd/pim_rpf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pimd/pim_rpf.c b/pimd/pim_rpf.c index b93f85e48..aa89431d3 100644 --- a/pimd/pim_rpf.c +++ b/pimd/pim_rpf.c @@ -262,7 +262,7 @@ enum pim_rpf_result pim_rpf_update(struct pim_instance *pim, if ((up->sg.src.s_addr == INADDR_ANY && I_am_RP(pim, up->sg.grp)) || PIM_UPSTREAM_FLAG_TEST_FHR(up->flags)) neigh_needed = false; - pim_find_or_track_nexthop(pim, &nht_p, up, NULL, false, NULL); + pim_find_or_track_nexthop(pim, &nht_p, up, NULL, NULL); if (!pim_ecmp_nexthop_lookup(pim, &rpf->source_nexthop, &src, &grp, neigh_needed)) { /* Route is Deleted in Zebra, reset the stored NH data */ |