ipn/ipnlocal: don't short-circuit default route filtering.

If no exit node is specified, the filter must still run to remove
offered default routes from all peers.

Signed-off-by: David Anderson <danderson@tailscale.com>
pull/1285/head
David Anderson 4 years ago
parent 1e7a35b225
commit e86b39b73f

@ -376,10 +376,6 @@ func (b *LocalBackend) setClientStatus(st controlclient.Status) {
// whether prefs was mutated as part of the process, due to an exit // whether prefs was mutated as part of the process, due to an exit
// node IP being converted into a node ID. // node IP being converted into a node ID.
func (b *LocalBackend) keepOneExitNodeLocked(nm *netmap.NetworkMap) (prefsChanged bool) { func (b *LocalBackend) keepOneExitNodeLocked(nm *netmap.NetworkMap) (prefsChanged bool) {
if b.prefs.ExitNodeID == "" && b.prefs.ExitNodeIP.IsZero() {
return false
}
// If we have a desired IP on file, try to find the corresponding // If we have a desired IP on file, try to find the corresponding
// node. // node.
if !b.prefs.ExitNodeIP.IsZero() { if !b.prefs.ExitNodeIP.IsZero() {

Loading…
Cancel
Save