diff options
| author | Enke Chen <enchen@paloaltonetworks.com> | 2025-01-14 19:29:22 +0100 |
|---|---|---|
| committer | Enke Chen <enchen@paloaltonetworks.com> | 2025-01-14 19:29:22 +0100 |
| commit | c467af10b2a97e8452b5b3bce245dbcae4ff21fa (patch) | |
| tree | da62dfbbd6b63fd6efaef8589e500b68c302e90f /bgpd/bgp_mplsvpn.c | |
| parent | Merge pull request #17800 from opensourcerouting/fix/import_between_vrf_respe... (diff) | |
| download | frr-c467af10b2a97e8452b5b3bce245dbcae4ff21fa.tar.xz frr-c467af10b2a97e8452b5b3bce245dbcae4ff21fa.zip | |
bgpd: move bgp_aggregate_increment() after bgp_path_info_add()
Route aggregation should be checked after a route is added, and
not before. This is for code flow and consistency.
Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>
Diffstat (limited to 'bgpd/bgp_mplsvpn.c')
| -rw-r--r-- | bgpd/bgp_mplsvpn.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bgpd/bgp_mplsvpn.c b/bgpd/bgp_mplsvpn.c index 1d10831ca..46e529f03 100644 --- a/bgpd/bgp_mplsvpn.c +++ b/bgpd/bgp_mplsvpn.c @@ -1312,8 +1312,8 @@ leak_update(struct bgp *to_bgp, struct bgp_dest *bn, else bgp_path_info_unset_flag(bn, new, BGP_PATH_VALID); - bgp_aggregate_increment(to_bgp, p, new, afi, safi); bgp_path_info_add(bn, new); + bgp_aggregate_increment(to_bgp, p, new, afi, safi); bgp_process(to_bgp, bn, new, afi, safi); |