summaryrefslogtreecommitdiffstats
path: root/bgpd/bgp_snmp.c
diff options
context:
space:
mode:
authorJosh Cox <josh.cox@pureport.com>2020-03-19 22:52:10 +0100
committerJosh Cox <josh.cox@pureport.com>2020-03-19 23:04:47 +0100
commitfd2e2db68040223c2851394e3791c433ece7ca16 (patch)
tree4bcebde3de3dec1fc9713b429cf00b2081ef4dc5 /bgpd/bgp_snmp.c
parentMerge pull request #6045 from rubenk/bgpd-use-correct-format-specifier (diff)
downloadfrr-fd2e2db68040223c2851394e3791c433ece7ca16.tar.xz
frr-fd2e2db68040223c2851394e3791c433ece7ca16.zip
bgpd: Fixed snmp and bmp 'just Established' test.
It was previously comparing an fsm event variable with an fsm status constant. This fixes issue #5963. Signed-off-by: Josh Cox <josh.cox@pureport.com>
Diffstat (limited to 'bgpd/bgp_snmp.c')
-rw-r--r--bgpd/bgp_snmp.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/bgpd/bgp_snmp.c b/bgpd/bgp_snmp.c
index d50716105..5cf0b7398 100644
--- a/bgpd/bgp_snmp.c
+++ b/bgpd/bgp_snmp.c
@@ -858,7 +858,7 @@ static int bgpTrapEstablished(struct peer *peer)
oid index[sizeof(oid) * IN_ADDR_SIZE];
/* Check if this peer just went to Established */
- if ((peer->last_major_event != OpenConfirm) || !(peer_established(peer)))
+ if ((peer->ostatus != OpenConfirm) || !(peer_established(peer)))
return 0;
ret = inet_aton(peer->host, &addr);