diff options
| author | Donatas Abraitis <donatas@opensourcerouting.org> | 2022-12-15 15:14:40 +0200 |
|---|---|---|
| committer | Donatas Abraitis <donatas@opensourcerouting.org> | 2022-12-19 23:27:33 +0200 |
| commit | 2b9d7b2c9b23be41a9e188402a3de0cbae95aa83 (patch) | |
| tree | adf5357c8e535e8a96b1c21843844bc69eeaaf62 | |
| parent | e3ba9ce36a8786337a9121d21db47ed97dc22da0 (diff) | |
bgpd: Make sure we have peer->notify.data before returning for SNMP
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
| -rw-r--r-- | bgpd/bgp_snmp_bgp4v2.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/bgpd/bgp_snmp_bgp4v2.c b/bgpd/bgp_snmp_bgp4v2.c index 2d70aa94d3..20854ff64e 100644 --- a/bgpd/bgp_snmp_bgp4v2.c +++ b/bgpd/bgp_snmp_bgp4v2.c @@ -367,9 +367,10 @@ static uint8_t *bgpv2PeerErrorsTable(struct variable *v, oid name[], } return SNMP_STRING(""); case BGP4V2_PEER_LAST_ERROR_SENT_DATA: - if (peer->last_reset == PEER_DOWN_NOTIFY_SEND || - peer->last_reset == PEER_DOWN_RTT_SHUTDOWN || - peer->last_reset == PEER_DOWN_USER_SHUTDOWN) + if ((peer->last_reset == PEER_DOWN_NOTIFY_SEND || + peer->last_reset == PEER_DOWN_RTT_SHUTDOWN || + peer->last_reset == PEER_DOWN_USER_SHUTDOWN) && + peer->notify.data) return SNMP_STRING(peer->notify.data); else return SNMP_STRING(""); |