]> git.puffer.fish Git - mirror/frr.git/commit
bgpd: fix flushing ipv6 flowspec entries when peering stops 15326/head
authorPhilippe Guibert <philippe.guibert@6wind.com>
Wed, 7 Feb 2024 21:34:34 +0000 (22:34 +0100)
committerPhilippe Guibert <philippe.guibert@6wind.com>
Wed, 7 Feb 2024 22:01:25 +0000 (23:01 +0100)
commitec6e09c27196c3006887a7a671521466cdcd1e4d
tree4c6a743a2c41048fb86176b5bd52fd4bb97f6619
parentab3d0846bd8893f2f93af1d3b93c2cb4ac6da0f0
bgpd: fix flushing ipv6 flowspec entries when peering stops

When a BGP flowspec peering stops, the BGP RIB entries for IPv6
flowspec entries are removed, but not the ZEBRA RIB IPv6 entries.

Actually, when calling bgp_zebra_withdraw() function call, only
the AFI_IP parameter is passed to the bgp_pbr_update_entry() function
in charge of the Flowspec add/delete in zebra. Fix this by passing
the AFI parameter to the bgp_zebra_withdraw() function.

Note that using topotest does not show up the problem as the
flowspec driver code is not present and was refused. Without that,
routes are not installed, and can not be uninstalled.

Fixes: 529efa234655 ("bgpd: allow flowspec entries to be announced to zebra")
Link: https://github.com/FRRouting/frr/pull/2025
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
bgpd/bgp_route.c
bgpd/bgp_zebra.c
bgpd/bgp_zebra.h