diff options
author | Steve Hill <quagga@cheesy.sackheads.org> | 2009-07-28 17:50:00 +0100 |
---|---|---|
committer | Paul Jakma <paul@quagga.net> | 2009-07-28 17:50:00 +0100 |
commit | 3117b5c47df044f8b2197fff60641075394fdce5 (patch) | |
tree | 0a25f9376ab96e21d9e161f9c9e156fbd5641d53 /lib/keychain.h | |
parent | cb9e0ceec988a059dd2269ebc54722c01afb7cca (diff) |
bgpd: fd leak in bgpd
* bgp_fsm.c: I have found an fd leak in bgpd that is caused by the 'new'
Clearing state. I've been seeing it from hold timer failures, but it can
also be triggered by other things.
When Hold_Timer_expired fires in Established state, a notify is sent and
BGP_Stop event queued. The fsm then transitions into Clearing state.
That is the problem; When the BGP_Stop event is serviced, the state table
says to ignore it while in Clearing. Thus bgp_stop is not called and the
fd leaks. Previously the peer would be in Idle state, which correctly
handles the BGP_Stop event.
Fix by making bgp_stop safe to call from Clearing state, without losing
ClearingCompleted events, and then ensuring it is called prior to
transition from Clearing->Idle.
Diffstat (limited to 'lib/keychain.h')
0 files changed, 0 insertions, 0 deletions