diff options
| author | Don Slice <dslice@cumulusnetworks.com> | 2016-11-23 11:58:27 -0800 |
|---|---|---|
| committer | Don Slice <dslice@cumulusnetworks.com> | 2017-05-17 08:18:14 -0400 |
| commit | 9c5cd62f08f5e0d77a6474e15a49dcf8bd8ce21c (patch) | |
| tree | 7ca8fc0896c181af81331bcd0b947f0d90db66ff /zebra/interface.c | |
| parent | 56b6674bc0f774168ea5e8330a7d8c3cb382d0e0 (diff) | |
zebra: Move interfaces to default before deleting
Encountered a crash in zebra due to getting a delete on an SVI with
VRR configured. Since we don't actually do a delete but flag the interface
as inactive, slag VRR interfaces would remain on the vrf_iflist with a lock
count of zero, causing the crash. Since all other interface types are moved
to the default table before deleting, doing the same thing for any interfaces
that were left in the vrf.
Testing includes manual testing, bgp-min, ospf-min, vrf-min, bgp-smoke, and ospf-smoke.
All passed (first time or on rerun) or match known failures.
Ticket: CM-13288
Signed-off-by: Don Slice
Reviewed-by: Donald Sharp
Diffstat (limited to 'zebra/interface.c')
| -rw-r--r-- | zebra/interface.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/zebra/interface.c b/zebra/interface.c index 39c20e6289..524183a434 100644 --- a/zebra/interface.c +++ b/zebra/interface.c @@ -693,6 +693,10 @@ if_delete_update (struct interface *ifp) for setting ifindex to IFINDEX_INTERNAL after processing the interface deletion message. */ ifp->ifindex = IFINDEX_INTERNAL; + + /* if the ifp is in a vrf, move it to default so vrf can be deleted if desired */ + if (ifp->vrf_id) + if_handle_vrf_change (ifp, VRF_DEFAULT); } /* VRF change for an interface */ |