diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2015-11-20 11:36:46 -0800 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2015-11-20 11:36:46 -0800 |
| commit | 1a1f4efab365954c7b0ab56e974c2d707c0923c5 (patch) | |
| tree | b54a9d3f085014bc38d6e1b21c39f703e7d1f4d5 /ospf6d/ospf6_zebra.c | |
| parent | f1121188c7c767d0ad45c02c852a984994f13223 (diff) | |
Quagga: vrf_id not being set correctly
Several routing protocols use the zapi_ipv[4|6] api to talk to
zebra. There are some instances where the api.vrf_id was not
being set. Since the practice is to declare the api structure
on the stack, the data inside is not being set to 0. As
such random vrf_id values were being passed to zebrad
causing rage and confusion.
Ticket: CM-8287
Reviewed-by: CCR-3841
Testing: Test suites no longer crashing and burning
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'ospf6d/ospf6_zebra.c')
| -rw-r--r-- | ospf6d/ospf6_zebra.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ospf6d/ospf6_zebra.c b/ospf6d/ospf6_zebra.c index b3e18bfd3e..7b756d152a 100644 --- a/ospf6d/ospf6_zebra.c +++ b/ospf6d/ospf6_zebra.c @@ -517,6 +517,7 @@ ospf6_zebra_add_discard (struct ospf6_route *request) { if (!CHECK_FLAG (request->flag, OSPF6_ROUTE_BLACKHOLE_ADDED)) { + api.vrf_id = VRF_DEFAULT; api.type = ZEBRA_ROUTE_OSPF6; api.flags = ZEBRA_FLAG_BLACKHOLE; api.instance = 0; @@ -561,7 +562,7 @@ ospf6_zebra_delete_discard (struct ospf6_route *request) { if (CHECK_FLAG (request->flag, OSPF6_ROUTE_BLACKHOLE_ADDED)) { - + api.vrf_id = VRF_DEFAULT; api.type = ZEBRA_ROUTE_OSPF6; api.flags = ZEBRA_FLAG_BLACKHOLE; api.instance = 0; |