ipn/ipnlocal: centralize assignments to cc + ccAuto in new method

cc vs ccAuto is a mess. It needs to go. But this is a baby step towards
getting there.

Updates #11649

Change-Id: I34f33934844e580bd823a7d8f2b945cf26c87b3b
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>

ipn/ipnlocal/local.go

@@ -1838,8 +1838,7 @@ func (b *LocalBackend) Start(opts ipn.Options) error {
 	if prevCC != nil {
 		defer prevCC.Shutdown() // must be called after b.mu is unlocked
 	}
-	b.cc = cc
+	b.setControlClientLocked(cc)
-	b.ccAuto, _ = cc.(*controlclient.Auto)
 	endpoints := b.endpoints
 
 	if err := b.initTKALocked(); err != nil {
@@ -4567,6 +4566,15 @@ func (b *LocalBackend) requestEngineStatusAndWait() {
 	b.statusLock.Unlock()
 }
 
+// setControlClientLocked sets the control client to cc,
+// which may be nil.
+//
+// b.mu must be held.
+func (b *LocalBackend) setControlClientLocked(cc controlclient.Client) {
+	b.cc = cc
+	b.ccAuto, _ = cc.(*controlclient.Auto)
+}
+
 // resetControlClientLocked sets b.cc to nil and returns the old value. If the
 // returned value is non-nil, the caller must call Shutdown on it after
 // releasing b.mu.
@@ -4589,8 +4597,7 @@ func (b *LocalBackend) resetControlClientLocked() controlclient.Client {
 		b.numClientStatusCalls.Add(1)
 	}
 	prev := b.cc
-	b.cc = nil
+	b.setControlClientLocked(nil)
-	b.ccAuto = nil
 	return prev
 }