app/vppn
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 9 Wiki Activity

Cleanup

Browse Source
This commit is contained in:
jdl
2024-12-24 19:41:07 +01:00
parent 6117a0afe4
commit b8d814f636
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
node/conn.go
View File

@@ -5,7 +5,6 @@ import (
"log" "log"
"net" "net"
"net/netip" "net/netip"
"runtime/debug"
"sync" "sync"
) )
@@ -21,9 +20,11 @@ func newConnWriter(conn *net.UDPConn) *connWriter {
} }
func (w *connWriter) WriteTo(packet []byte, addr netip.AddrPort) { func (w *connWriter) WriteTo(packet []byte, addr netip.AddrPort) {
// Even though a conn is safe for concurrent use, it turns out that a mutex
// in Go is more fair when there's contention. Without this lock, control
// packets may fail to be sent in a timely manner causing timeouts.
w.lock.Lock() w.lock.Lock()
if _, err := w.conn.WriteToUDPAddrPort(packet, addr); err != nil { if _, err := w.conn.WriteToUDPAddrPort(packet, addr); err != nil {
debug.PrintStack()
log.Fatalf("Failed to write to UDP port: %v", err) log.Fatalf("Failed to write to UDP port: %v", err)
} }
w.lock.Unlock() w.lock.Unlock()