From: Martin Buck Date: Mon, 17 Jan 2022 12:54:02 +0000 (+0100) Subject: ospf6d: Fix ECMP inter-area route nexthop update X-Git-Tag: base_8.2~6^2 X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=b74e965142d80b7827cd6ac1291e1a3a73b9e617;p=mirror%2Ffrr.git ospf6d: Fix ECMP inter-area route nexthop update Fixes #9720. When updating an ECMP inter-area route, we compute a new route and check whether that already exists. If so, we keep the old route and only update its nexthops. Previously, we merged the new route's nexthops into the old one's, but this way, it's impossible to remove nexthops from the old route, resulting in stale nexthops. This commit fixes this by first removing all nexthops from the old route and then copying all nexthops from the new route into it. If the new route has fewer nexthops, the old one will have as well afterwards. Signed-off-by: Martin Buck --- diff --git a/ospf6d/ospf6_abr.c b/ospf6d/ospf6_abr.c index 443759344e..5af1139d9b 100644 --- a/ospf6d/ospf6_abr.c +++ b/ospf6d/ospf6_abr.c @@ -1276,7 +1276,8 @@ void ospf6_abr_examin_summary(struct ospf6_lsa *lsa, struct ospf6_area *oa) continue; } - ospf6_route_merge_nexthops(old_route, route); + list_delete_all_node(old_route->nh_list); + ospf6_route_copy_nexthops(old_route, route); old_entry_updated = true; for (ALL_LIST_ELEMENTS_RO(old_route->paths, anode,