Merge pull request #17394 from opensourcerouting/fix/backport_17376_9.1

bgpd: Clear stale routes with multiple paths (backport)
FRRouting · Nov 8, 2024 · a127099 · a127099
2 parents dd3e867 + 875337f
commit a127099
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/bgpd/bgp_fsm.c b/bgpd/bgp_fsm.c
@@ -685,6 +685,11 @@ static void bgp_set_llgr_stale(struct peer *peer, afi_t afi, safi_t safi)
 						    COMMUNITY_NO_LLGR))
 						continue;
 
+					if (bgp_attr_get_community(pi->attr) &&
+					    community_include(bgp_attr_get_community(pi->attr),
+							      COMMUNITY_LLGR_STALE))
+						continue;
+
 					if (bgp_debug_neighbor_events(peer))
 						zlog_debug(
 							"%pBP Long-lived set stale community (LLGR_STALE) for: %pFX",
@@ -695,8 +700,6 @@ static void bgp_set_llgr_stale(struct peer *peer, afi_t afi, safi_t safi)
 					pi->attr = bgp_attr_intern(&attr);
 					bgp_recalculate_afi_safi_bestpaths(
 						peer->bgp, afi, safi);
-
-					break;
 				}
 		}
 	} else {
@@ -713,6 +716,11 @@ static void bgp_set_llgr_stale(struct peer *peer, afi_t afi, safi_t safi)
 					    COMMUNITY_NO_LLGR))
 					continue;
 
+				if (bgp_attr_get_community(pi->attr) &&
+				    community_include(bgp_attr_get_community(pi->attr),
+						      COMMUNITY_LLGR_STALE))
+					continue;
+
 				if (bgp_debug_neighbor_events(peer))
 					zlog_debug(
 						"%pBP Long-lived set stale community (LLGR_STALE) for: %pFX",
@@ -723,8 +731,6 @@ static void bgp_set_llgr_stale(struct peer *peer, afi_t afi, safi_t safi)
 				pi->attr = bgp_attr_intern(&attr);
 				bgp_recalculate_afi_safi_bestpaths(peer->bgp,
 								   afi, safi);
-
-				break;
 			}
 	}
 }

diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c
@@ -5856,7 +5856,6 @@ void bgp_clear_stale_route(struct peer *peer, afi_t afi, safi_t safi)
 						vpn_leak_to_vrf_withdraw(pi);
 
 					bgp_rib_remove(rm, pi, peer, afi, safi);
-					break;
 				}
 		}
 	} else {
@@ -5885,7 +5884,6 @@ void bgp_clear_stale_route(struct peer *peer, afi_t afi, safi_t safi)
 						pi);
 
 				bgp_rib_remove(dest, pi, peer, afi, safi);
-				break;
 			}
 	}
 }
-Original file line number
+Diff line change
@@ Expand Up @@
     						vpn_leak_to_vrf_withdraw(pi);
     					bgp_rib_remove(rm, pi, peer, afi, safi);
-    					break;
     				}
     		}
     	} else {
@@ Expand Down Expand Up @@
     						pi);
     				bgp_rib_remove(dest, pi, peer, afi, safi);
-    				break;
     			}
     	}
     }
@@ Expand Down @@