vet: nil ptr auto.go

ignoramous · ignoramous · commit 6e793da4b0bf · 2026-07-03T03:01:48.000+05:30
diff --git a/intra/ipn/auto.go b/intra/ipn/auto.go
@@ -580,7 +580,10 @@ func laddr(c net.Conn) (string, bool) {
 	if c == nil || core.IsNil(c) {
 		return "", false
 	}
-	return c.LocalAddr().String(), true
+	if addr := c.LocalAddr(); addr != nil {
+		return addr.String(), true
+	}
+	return "", false
 }
 
 func maybeKeepAlive(c net.Conn) (keepingalive bool) {
diff --git a/intra/ipn/wgproxy.go b/intra/ipn/wgproxy.go
@@ -123,6 +123,7 @@ type wgtun struct {
 
 	// mutable fields
 
+	// atomic.Pointer over different Proxy impls will work go.dev/play/p/2ovWbpkWO1f
 	via    atomic.Pointer[core.WeakRef[Proxy]]
 	viaUp  atomic.Bool // using via?
 	direct protect.RDialer

Original file line number	Diff line number	Diff line change
`@@ -580,7 +580,10 @@ func laddr(c net.Conn) (string, bool) {`
`580`	`580`	`if c == nil \|\| core.IsNil(c) {`
`581`	`581`	`return "", false`
`582`	`582`	`}`
`583`		`- return c.LocalAddr().String(), true`
	`583`	`+ if addr := c.LocalAddr(); addr != nil {`
	`584`	`+ return addr.String(), true`
	`585`	`+ }`
	`586`	`+ return "", false`
`584`	`587`	`}`
`585`	`588`
`586`	`589`	`func maybeKeepAlive(c net.Conn) (keepingalive bool) {`