controlclient: do not send duplicate hostinfo/netinfocrawshaw/hostinfo

This should never happen, so log when it does so we can fix it.

Signed-off-by: David Crawshaw <crawshaw@tailscale.com>

1 files changed, 12 insertions, 2 deletions

diff --git a/control/controlclient/auto.go b/control/controlclient/auto.go
index ce49ad0ec..831b736cb 100644
--- a/control/controlclient/auto.go
+++ b/control/controlclient/auto.go
@@ -496,7 +496,12 @@ func (c *Client) SetHostinfo(hi *tailcfg.Hostinfo) {
 	if hi == nil {
 		panic("nil Hostinfo")
 	}
-	c.direct.SetHostinfo(hi)
+	if !c.direct.SetHostinfo(hi) {
+		c.logf("[unexpected] duplicate Hostinfo: %v", hi)
+		return
+	}
+	c.logf("Hostinfo: %v", hi)
+
 	// Send new Hostinfo to server
 	c.cancelMapSafely()
 }
@@ -505,7 +510,12 @@ func (c *Client) SetNetInfo(ni *tailcfg.NetInfo) {
 	if ni == nil {
 		panic("nil NetInfo")
 	}
-	c.direct.SetNetInfo(ni)
+	if !c.direct.SetNetInfo(ni) {
+		c.logf("[unexpected] duplicate NetInfo: %v", ni)
+		return
+	}
+	c.logf("NetInfo: %v", ni)
+
 	// Send new Hostinfo (which includes NetInfo) to server
 	c.cancelMapSafely()
 }