mirror of
https://git.proxmox.com/git/mirror_frr
synced 2025-08-05 04:40:21 +00:00
bgpd: fix for leaked struct bgp_adj_[in|out] on peer shutdown
If a peer with soft-reconfiguration configured is cleared, the function bgp_clear_route_table() doesn't free the bgp_adj_in and bgp_adj_out structures of route nodes that for some reason, ej. denied by a filter, don't have routes attached "rn->info == NULL". Signed-off-by: Jorge Boncompte [DTI2] <jorge@dti2.net> Reviewed-by: Leonid Rosenboim <Leonid.Rosenboim@windriver.com> Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
This commit is contained in:
parent
a5c851c7ff
commit
ebbb5fca5c
@ -2810,9 +2810,6 @@ bgp_clear_route_table (struct peer *peer, afi_t afi, safi_t safi,
|
||||
struct bgp_info *ri;
|
||||
struct bgp_adj_in *ain;
|
||||
struct bgp_adj_out *aout;
|
||||
|
||||
if (rn->info == NULL)
|
||||
continue;
|
||||
|
||||
/* XXX:TODO: This is suboptimal, every non-empty route_node is
|
||||
* queued for every clearing peer, regardless of whether it is
|
||||
|
Loading…
Reference in New Issue
Block a user