From 8d5ed65e0d8f8a06c0a3ba900996c7007020d6fd Mon Sep 17 00:00:00 2001 From: Donald Sharp Date: Sat, 20 Jan 2024 21:32:36 -0500 Subject: [PATCH] zebra: Cleanup dest assignment dest was shadowing dest inside of an if statement additionally both legs needed dest to be assigned. Let's clean this up a slight bit and use it appropriately Signed-off-by: Donald Sharp --- zebra/zebra_rib.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/zebra/zebra_rib.c b/zebra/zebra_rib.c index 4b5f81a3df..9e89e32d5d 100644 --- a/zebra/zebra_rib.c +++ b/zebra/zebra_rib.c @@ -2874,11 +2874,10 @@ static void process_subq_early_route_add(struct zebra_early_route *ere) SET_FLAG(re->status, ROUTE_ENTRY_CHANGED); rib_addnode(rn, re, 1); + dest = rib_dest_from_rnode(rn); /* Free implicit route.*/ if (same) { - rib_dest_t *dest = rn->info; - - if (same == dest->selected_fib) + if (dest && same == dest->selected_fib) SET_FLAG(same->status, ROUTE_ENTRY_ROUTE_REPLACING); rib_delnode(rn, same); } @@ -2886,7 +2885,6 @@ static void process_subq_early_route_add(struct zebra_early_route *ere) /* See if we can remove some RE entries that are queued for * removal, but won't be considered in rib processing. */ - dest = rib_dest_from_rnode(rn); RNODE_FOREACH_RE_SAFE (rn, re, same) { if (CHECK_FLAG(re->status, ROUTE_ENTRY_REMOVED)) { /* If the route was used earlier, must retain it. */ -- 2.39.5