Debug 2

Signed-off-by: Maxence Younsi <[email protected]>

bgpd/bgp_route.c

@@ -3408,6 +3408,9 @@ static void bgp_process_main_one(struct bgp *bgp, struct bgp_dest *dest,
 	 * set
 	 */
 	if (old_select || new_select) {
+		if (old_select)
+			bgp_path_info_lock(old_select, __func__);
+
 		hook_call(bgp_route_update, bgp, afi, safi, dest, old_select,
 			  new_select);
 	}
@@ -3589,6 +3592,7 @@ static void bgp_process_main_one(struct bgp *bgp, struct bgp_dest *dest,
 
 	/* Reap old select bgp_path_info, if it has been removed */
 	if (old_select && CHECK_FLAG(old_select->flags, BGP_PATH_REMOVED)) {
+		zlog_info("bgp reap old path %p", old_select);
 		bgp_path_info_reap(dest, old_select);
 	}
 
@@ -3599,6 +3603,7 @@ static void bgp_process_main_one(struct bgp *bgp, struct bgp_dest *dest,
 			old_select->peer->stat_loc_rib_count[afi][safi]--;
 
 		hook_call(bgp_process_main_one_end, bgp, old_select);
+		bgp_path_info_unlock(old_select, __func__);
 	}
 
 	if (new_select && new_select->peer)
@@ -3619,7 +3624,6 @@ static void bgp_process_main_one(struct bgp *bgp, struct bgp_dest *dest,
 	}
 	bgp_mpath_diff_clear(&mpath_diff);
 	bgp_mpath_diff_fini(&mpath_diff);
-	return;
 }
 
 /* Process the routes with the flag BGP_NODE_SELECT_DEFER set */