ipn/ipnlocal: fix data race
We can't access b.netMap without holding b.mu. We already grabbed it earlier in the function with the lock held. Introduced in Nov 2020 in7ea809897d
. Discovered during stress testing. Apparently it's a pretty rare? Signed-off-by: Josh Bleecher Snyder <josh@tailscale.com> (cherry picked from commitf11a8928a6
)
This commit is contained in:
parent
405ea978f8
commit
506476059a
|
@ -2213,7 +2213,7 @@ func (b *LocalBackend) enterState(newState ipn.State) {
|
|||
b.e.RequestStatus()
|
||||
case ipn.Running:
|
||||
var addrs []string
|
||||
for _, addr := range b.netMap.Addresses {
|
||||
for _, addr := range netMap.Addresses {
|
||||
addrs = append(addrs, addr.IP().String())
|
||||
}
|
||||
systemd.Status("Connected; %s; %s", activeLogin, strings.Join(addrs, " "))
|
||||
|
|
Loading…
Reference in New Issue