Cleanup AID handling.
authorclaudio <claudio@openbsd.org>
Wed, 20 Mar 2024 09:35:46 +0000 (09:35 +0000)
committerclaudio <claudio@openbsd.org>
Wed, 20 Mar 2024 09:35:46 +0000 (09:35 +0000)
commit110c1584e6f06f8c1d2e120f71e691b8225c4c2f
tree763e1304c8f89eff4cef2dae9b7c6a90236ce8e9
parent81b5c8f98a2a1ff4a013450db6d29b7a72829fcd
Cleanup AID handling.

- Loops over all valid AID should start with AID_MIN and go up to AID_MAX - 1
   e.g. for (i = AID_MIN; i < AID_MAX; i++)
  If for some reason AID_UNSPEC must be handled make that explicit in the
  for loop.

- aid2afi() now returns an error for AID_UNSPEC since there is no valid
  AFI SAFI combo for AID_UNSPEC.

- Add additional checks for AID_MIN where currently only AID_MAX was checked.
  This affects imsg for route refresh and graceful restart.

- Simplify add-path capability handling. Only the negotiated add_path capa
  sets the flag for AID_UNSPEC to help code to quickly check if any add-path
  is active.

OK tb@
usr.sbin/bgpd/parse.y
usr.sbin/bgpd/printconf.c
usr.sbin/bgpd/rde.c
usr.sbin/bgpd/rde_peer.c
usr.sbin/bgpd/session.c
usr.sbin/bgpd/util.c