stunner: fix data race.
In very low-latency conditions, a STUN request can complete before
the startup loop has finished firing off goroutines, leading to
a concurrent map mutation.
@@ -184,6 +184,7 @@ type sender struct {
defer cancel()
need[server] = sender{ctx, cancel}
}
+ needMu.Lock()
for server, sender := range need {
wg.Add(1)
server, ctx := server, sender.ctx
@@ -192,6 +193,7 @@ type sender struct {
s.sendPackets(ctx, server)
}()
}
+ needMu.Unlock()
var err error
select {
case <-ctx.Done():
