Skip to content

Commit 422c3ae

Browse files
authored
Fix up some BGP undefined variables (librenms#17698)
1 parent 989d377 commit 422c3ae

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

includes/polling/bgp-peers.inc.php

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,7 @@
353353
} elseif ($device['os'] == 'aos7') {
354354
$peer_identifier = $peer['bgpPeerIdentifier'];
355355
$peer_data = [];
356-
$al_descr = snmpwalk_cache_multi_oid($device, 'alaBgpPeerName', $al_descr, 'ALCATEL-IND1-BGP-MIB', 'aos7', '-OQUs');
356+
$al_descr = snmpwalk_cache_multi_oid($device, 'alaBgpPeerName', [], 'ALCATEL-IND1-BGP-MIB', 'aos7', '-OQUs');
357357
$al_peer = snmpwalk_cache_multi_oid($device, 'BgpPeerEntry', [], 'BGP4-MIB', 'aos7', '-OQUs');
358358
$peer_data['bgpPeerDescr'] = $al_descr[$peer_identifier]['alaBgpPeerName'];
359359
$peer_data['bgpPeerState'] = $al_peer[$peer_identifier]['bgpPeerState'];
@@ -669,8 +669,8 @@
669669
} else {
670670
$tmp_peer = $peer['bgpPeerIdentifier'];
671671
}
672-
$a_prefixes = snmpwalk_cache_multi_oid($device, 'aristaBgp4V2PrefixInPrefixesAccepted', $a_prefixes, 'ARISTA-BGP4V2-MIB', null, '-OQUs');
673-
$out_prefixes = snmpwalk_cache_multi_oid($device, 'aristaBgp4V2PrefixOutPrefixes', $out_prefixes, 'ARISTA-BGP4V2-MIB', null, '-OQUs');
672+
$a_prefixes = snmpwalk_cache_multi_oid($device, 'aristaBgp4V2PrefixInPrefixesAccepted', [], 'ARISTA-BGP4V2-MIB', null, '-OQUs');
673+
$out_prefixes = snmpwalk_cache_multi_oid($device, 'aristaBgp4V2PrefixOutPrefixes', [], 'ARISTA-BGP4V2-MIB', null, '-OQUs');
674674

675675
$cbgpPeerAcceptedPrefixes = $a_prefixes["1.$afi.$tmp_peer.$afi.$safi"]['aristaBgp4V2PrefixInPrefixesAccepted'];
676676
$cbgpPeerAdvertisedPrefixes = $out_prefixes["1.$afi.$tmp_peer.$afi.$safi"]['aristaBgp4V2PrefixOutPrefixes'];
@@ -697,7 +697,7 @@
697697

698698
if ($device['os'] === 'aos7') {
699699
$tmp_peer = $peer['bgpPeerIdentifier'];
700-
$al_prefixes = snmpwalk_cache_multi_oid($device, 'alaBgpPeerRcvdPrefixes', $al_prefixes, 'ALCATEL-IND1-BGP-MIB', 'aos7', '-OQUs');
700+
$al_prefixes = snmpwalk_cache_multi_oid($device, 'alaBgpPeerRcvdPrefixes', [], 'ALCATEL-IND1-BGP-MIB', 'aos7', '-OQUs');
701701
$cbgpPeerAcceptedPrefixes = $al_prefixes[$tmp_peer]['alaBgpPeerRcvdPrefixes'];
702702
}
703703

@@ -737,14 +737,14 @@
737737
}
738738

739739
// Validate data
740-
$cbgpPeerAcceptedPrefixes = set_numeric($cbgpPeerAcceptedPrefixes);
741-
$cbgpPeerDeniedPrefixes = set_numeric($cbgpPeerDeniedPrefixes);
742-
$cbgpPeerPrefixAdminLimit = set_numeric($cbgpPeerPrefixAdminLimit);
743-
$cbgpPeerPrefixThreshold = set_numeric($cbgpPeerPrefixThreshold);
744-
$cbgpPeerPrefixClearThreshold = set_numeric($cbgpPeerPrefixClearThreshold);
745-
$cbgpPeerAdvertisedPrefixes = set_numeric($cbgpPeerAdvertisedPrefixes);
746-
$cbgpPeerSuppressedPrefixes = set_numeric($cbgpPeerSuppressedPrefixes);
747-
$cbgpPeerWithdrawnPrefixes = set_numeric($cbgpPeerWithdrawnPrefixes);
740+
$cbgpPeerAcceptedPrefixes = set_numeric($cbgpPeerAcceptedPrefixes ?? 0);
741+
$cbgpPeerDeniedPrefixes = set_numeric($cbgpPeerDeniedPrefixes ?? 0);
742+
$cbgpPeerPrefixAdminLimit = set_numeric($cbgpPeerPrefixAdminLimit ?? 0);
743+
$cbgpPeerPrefixThreshold = set_numeric($cbgpPeerPrefixThreshold ?? 0);
744+
$cbgpPeerPrefixClearThreshold = set_numeric($cbgpPeerPrefixClearThreshold ?? 0);
745+
$cbgpPeerAdvertisedPrefixes = set_numeric($cbgpPeerAdvertisedPrefixes ?? 0);
746+
$cbgpPeerSuppressedPrefixes = set_numeric($cbgpPeerSuppressedPrefixes ?? 0);
747+
$cbgpPeerWithdrawnPrefixes = set_numeric($cbgpPeerWithdrawnPrefixes ?? 0);
748748

749749
$cbgpPeers_cbgp_fields = [
750750
'AcceptedPrefixes' => $cbgpPeerAcceptedPrefixes,

0 commit comments

Comments
 (0)