summaryrefslogtreecommitdiffstats
path: root/bgpd/bgp_packet.c
diff options
context:
space:
mode:
authorDonald Sharp <sharpd@cumulusnetworks.com>2018-03-21 00:02:19 +0100
committerDonald Sharp <sharpd@cumulusnetworks.com>2018-03-21 00:09:06 +0100
commit5410015a79405ea53517257d443870f9f8bcf9b9 (patch)
tree4871f092755d0b5d46a0ca9905aa5c86f5be02f2 /bgpd/bgp_packet.c
parentbgpd: More ecommunity string leak cleanup (diff)
downloadfrr-5410015a79405ea53517257d443870f9f8bcf9b9.tar.xz
frr-5410015a79405ea53517257d443870f9f8bcf9b9.zip
bgpd: peer->bgp must be non NULL
We lock and set peer->bgp at peer creation and only remove it at deletion. Therefore these tests are not needed. Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'bgpd/bgp_packet.c')
-rw-r--r--bgpd/bgp_packet.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/bgpd/bgp_packet.c b/bgpd/bgp_packet.c
index cb702d80d..d17c33441 100644
--- a/bgpd/bgp_packet.c
+++ b/bgpd/bgp_packet.c
@@ -389,7 +389,7 @@ int bgp_generate_updgrp_packets(struct thread *thread)
if (peer->status != Established)
return 0;
- if (peer->bgp && peer->bgp->main_peers_update_hold)
+ if (peer->bgp->main_peers_update_hold)
return 0;
do {