summaryrefslogtreecommitdiff
path: root/zebra
diff options
context:
space:
mode:
authorMark Stapp <mjs@cisco.com>2025-01-27 14:17:24 -0500
committerMark Stapp <mjs@cisco.com>2025-01-27 14:17:24 -0500
commitcb7cf73992847cfd4af796085bf14f2fdc4fa8db (patch)
treebdc6e52de34acffffd24851fd6ce544403b32a12 /zebra
parent9890d3acce8014e0eee6cfa40295b6866c5defb0 (diff)
zebra: include resolving nexthops in nhg hash
Ensure that the nhg hash comparison function includes all nexthops, including recursive-resolving nexthops. Signed-off-by: Mark Stapp <mjs@cisco.com>
Diffstat (limited to 'zebra')
-rw-r--r--zebra/zebra_nhg.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/zebra/zebra_nhg.c b/zebra/zebra_nhg.c
index 4f6bc02c6e..f5141c8f23 100644
--- a/zebra/zebra_nhg.c
+++ b/zebra/zebra_nhg.c
@@ -572,8 +572,7 @@ bool zebra_nhg_hash_equal(const void *arg1, const void *arg2)
/* Nexthops should be in-order, so we simply compare them in-place */
for (nexthop1 = nhe1->nhg.nexthop, nexthop2 = nhe2->nhg.nexthop;
nexthop1 && nexthop2;
- nexthop1 = nexthop1->next, nexthop2 = nexthop2->next) {
-
+ nexthop1 = nexthop_next(nexthop1), nexthop2 = nexthop_next(nexthop2)) {
if (!nhg_compare_nexthops(nexthop1, nexthop2))
return false;
}
@@ -608,8 +607,7 @@ bool zebra_nhg_hash_equal(const void *arg1, const void *arg2)
for (nexthop1 = nhe1->backup_info->nhe->nhg.nexthop,
nexthop2 = nhe2->backup_info->nhe->nhg.nexthop;
nexthop1 && nexthop2;
- nexthop1 = nexthop1->next, nexthop2 = nexthop2->next) {
-
+ nexthop1 = nexthop_next(nexthop1), nexthop2 = nexthop_next(nexthop2)) {
if (!nhg_compare_nexthops(nexthop1, nexthop2))
return false;
}