diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-05-31 15:37:34 +0200 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-05-31 15:37:34 +0200 |
| commit | f220da9968741598ef224cdaf66a131026498a64 (patch) | |
| tree | 314f6253dc19b0882a780b847545cf5499fa165f /bgpd/bgp_zebra.c | |
| parent | bgpd: Add null0 route back for aggregate-address route created. (diff) | |
| download | frr-f220da9968741598ef224cdaf66a131026498a64.tar.xz frr-f220da9968741598ef224cdaf66a131026498a64.zip | |
bgpd: Fix SA issue with uninited ifindex value
The rearrangement of where the decision point of
filling in the aggregate information, must have allowed
SA to find a path of code where we may use ifindex uninitialized.
While I don't think this is possible to happen, make this issue
go away.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'bgpd/bgp_zebra.c')
| -rw-r--r-- | bgpd/bgp_zebra.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bgpd/bgp_zebra.c b/bgpd/bgp_zebra.c index e4f4823e6..1cd51cb6f 100644 --- a/bgpd/bgp_zebra.c +++ b/bgpd/bgp_zebra.c @@ -1316,7 +1316,7 @@ void bgp_zebra_announce(struct bgp_node *rn, struct prefix *p, &mpinfo_cp->attr->nexthop, mpinfo_cp->attr, is_evpn, api_nh); } else { - ifindex_t ifindex; + ifindex_t ifindex = IFINDEX_INTERNAL; struct in6_addr *nexthop; if (bgp->table_map[afi][safi].name) { |