
equinox at opensourcerouting
Jun 21, 2012, 1:02 AM
Post #1 of 1
(139 views)
Permalink
|
|
[PATCH 1/3] bgpd: Fix for commit 6a4677b7, fixes BZ#729
|
|
From: "Jorge Boncompte [DTI2]" <jorge [at] dti2> The timers are rearmed after events processing. After 6a4677b7 we do not generate events that can rearm the holdtime timer. Fix it's to call bgp_timer_set() directly as it's done from bgp_event(). Signed-off-by: Jorge Boncompte [DTI2] <jorge [at] dti2> Tested-by: Martin Winter <mwinter [at] opensourcerouting> Signed-off-by: David Lamparter <equinox [at] opensourcerouting> --- bgpd/bgp_packet.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/bgpd/bgp_packet.c b/bgpd/bgp_packet.c index 024f5e7..2d62c8d 100644 --- a/bgpd/bgp_packet.c +++ b/bgpd/bgp_packet.c @@ -1854,8 +1854,9 @@ bgp_update_receive (struct peer *peer, bgp_size_t size) peer->update_in++; peer->update_time = bgp_clock (); - /* Cancel holdtime timer */ + /* Rearm holdtime timer */ BGP_TIMER_OFF (peer->t_holdtime); + bgp_timer_set (peer); return 0; } -- 1.7.8.6 _______________________________________________ Quagga-dev mailing list Quagga-dev [at] lists http://lists.quagga.net/mailman/listinfo/quagga-dev
|