summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDonald Sharp <sharpd@nvidia.com>2020-12-04 08:01:31 -0500
committerDonald Sharp <sharpd@nvidia.com>2020-12-04 08:01:31 -0500
commit06f16b2c253ff9d71db498e3a93a0d620434496c (patch)
treed7bf04a544fb82cdb073c6d60bdb46813b455d0e
parent057edd2e84a6536615b4c80c67d83024ef8f3234 (diff)
bgpd: Let's actually track if the nh was updated
In bgp_zebra_announce when iterating over multipath we were checking to ensure that the nexthop was updated but never initially clearing the nh_updated variable. Thus leading to a situation where we could crash. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
-rw-r--r--bgpd/bgp_zebra.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/bgpd/bgp_zebra.c b/bgpd/bgp_zebra.c
index 12bc2e41bc..8d07e050f8 100644
--- a/bgpd/bgp_zebra.c
+++ b/bgpd/bgp_zebra.c
@@ -1176,7 +1176,7 @@ void bgp_zebra_announce(struct bgp_dest *dest, const struct prefix *p,
int nh_othervrf = 0;
char buf_prefix[PREFIX_STRLEN]; /* filled in if we are debugging */
bool is_evpn;
- int nh_updated;
+ bool nh_updated = false;
bool do_wt_ecmp;
uint64_t cum_bw = 0;
uint32_t nhg_id = 0;