Merge pull request #18562 from opensourcerouting/fix/bfd_down_if_established

mjstapp · web-flow · commit 259ffe1dfe0c · 2025-04-04T12:28:18.000-04:00
bgpd: Treat the peer as not active due to BFD down only if established
diff --git a/bgpd/bgpd.c b/bgpd/bgpd.c
@@ -4789,7 +4789,7 @@ enum bgp_peer_active peer_active(struct peer_connection *connection)
 		return BGP_PEER_CONNECTION_UNSPECIFIED;
 
 	if (peer->bfd_config) {
-		if (bfd_session_is_down(peer->bfd_config->session))
+		if (peer_established(connection) && bfd_session_is_down(peer->bfd_config->session))
 			return BGP_PEER_BFD_DOWN;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -4789,7 +4789,7 @@ enum bgp_peer_active peer_active(struct peer_connection *connection)`
`4789`	`4789`	`return BGP_PEER_CONNECTION_UNSPECIFIED;`
`4790`	`4790`
`4791`	`4791`	`if (peer->bfd_config) {`
`4792`		`- if (bfd_session_is_down(peer->bfd_config->session))`
	`4792`	`+ if (peer_established(connection) && bfd_session_is_down(peer->bfd_config->session))`
`4793`	`4793`	`return BGP_PEER_BFD_DOWN;`
`4794`	`4794`	`}`
`4795`	`4795`