bgpd: fix missing addpath withdrawal race condition

There is a race condition with addpath withdrawal. The withdrawal never
happens if it was request when coalesce timer running.

It can be demonstrated by adding to bgp_snmp_bgp4v2mib/rr/bgpd.conf:

> router bgp 65004
> + coalesce-time 10000

But it also happens in other conditions. For example, when using
"vtysh -f" to load the configuration.

It results in having two identical paths but with different addpath on
r2:

> r2# sh bgp ipv4 10.0.0.0/31
> BGP routing table entry for 10.0.0.0/31, version 3
> Paths: (3 available, best #1, table default)
>   65004
>     192.168.12.4 from 192.168.12.4 (192.168.12.4)
>       Origin IGP, metric 1, valid, external, multipath, best (AS Path)
>       AddPath ID: RX 0, TX-All 2 TX-Best-Per-AS 0 TX-Best-Selected 0
>       Advertised to: 192.168.12.4
>       Last update: Thu Sep 12 16:13:59 2024
>   65004
>     192.168.12.4 from 192.168.12.4 (192.168.12.4)
>       Origin IGP, metric 1, valid, external, multipath
>       AddPath ID: RX 3, TX-All 4 TX-Best-Per-AS 0 TX-Best-Selected 0
>       Advertised to: 192.168.12.4
>       Last update: Thu Sep 12 16:13:59 2024

The first path has been created first but should be withdrawn later.

Withdraw the stale addpath even the coalesce timer is running.

Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>

0 files changed, 0 insertions, 0 deletions