From 2e1bc6e500be4b7bdec3304245c8d19ab186a97c Mon Sep 17 00:00:00 2001 From: Joseph Henry Date: Tue, 9 Jan 2018 09:59:55 -0800 Subject: [PATCH] Cleaned up old code for attempted fix for ticket #600 --- osdep/ManagedRoute.cpp | 25 ------------------------- service/OneService.cpp | 1 + 2 files changed, 1 insertion(+), 25 deletions(-) diff --git a/osdep/ManagedRoute.cpp b/osdep/ManagedRoute.cpp index 8ffbed2ac..d7c807049 100644 --- a/osdep/ManagedRoute.cpp +++ b/osdep/ManagedRoute.cpp @@ -477,31 +477,6 @@ bool ManagedRoute::sync() } } - // Detect routes previously applied by ZT that don't exist in the system's route list. Re-apply these. - // NOTE: The following block was added in reaction to the macOS High Sierra 10.13.2 disappearing - // route issue. This comment should be removed once we're sure this block doesn't have any side-effects. - /* - - // Commented out temporarily to test more generalized fix - - bool found = false; - std::vector<_RTE> currRoutes(_getRTEs(_target,false)); - for(std::vector<_RTE>::iterator r(currRoutes.begin());r!=currRoutes.end();++r) { - if(_target == r->target) { - found = true; - break; - } - } - if (!found) { - // erase _applied enteries - std::map::iterator rt; - rt = _applied.find(leftt); - _applied.erase(rt, _applied.end()); - rt = _applied.find(rightt); - _applied.erase(rt, _applied.end()); - } - */ - if (!_applied.count(leftt)) { _applied[leftt] = false; // not ifscoped _routeCmd("add",leftt,_via,(const char *)0,(_via) ? (const char *)0 : _device); diff --git a/service/OneService.cpp b/service/OneService.cpp index e65ad0e23..92b78a8cc 100644 --- a/service/OneService.cpp +++ b/service/OneService.cpp @@ -1609,6 +1609,7 @@ public: bool haveRoute = false; // Ignore routes implied by local managed IPs since adding the IP adds the route + // Commented out to fix ticket #600 (disappearing routes on macOS). Remove this block when we're sure there's no side effects /* for(std::vector::iterator ip(n.managedIps.begin());ip!=n.managedIps.end();++ip) { if ((target->netmaskBits() == ip->netmaskBits())&&(target->containsAddress(*ip))) {