From: qmuntal <quimmuntal@gmail.com>
Date: Fri, 12 Sep 2025 08:13:15 +0000 (+0200)
Subject: net,internal/poll: skip TestAllocs when race is enabled on Windows
X-Git-Tag: go1.26rc1~892
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=a4e25c3d657cf7c3f3a921cc99ea60ad5a337b82;p=gostls13.git

net,internal/poll: skip TestAllocs when race is enabled on Windows

The Windows implementation of several network protocols make use of
sync.Pool, which randomly drops cached items when race is enabled.

While here, zero out the control buffer to allow it to be garbage
collected.

Fixes #75341

Change-Id: Ie20e21adef2edc02ca7b4a78012dd5f3a9f03bee
Reviewed-on: https://go-review.googlesource.com/c/go/+/703195
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
---

diff --git a/src/internal/poll/fd_windows.go b/src/internal/poll/fd_windows.go
index f8d41bafd1..23500d3e39 100644
--- a/src/internal/poll/fd_windows.go
+++ b/src/internal/poll/fd_windows.go
@@ -196,6 +196,8 @@ func freeWSAMsg(msg *windows.WSAMsg) {
 	// Clear pointers to buffers so they can be released by garbage collector.
 	msg.Buffers.Len = 0
 	msg.Buffers.Buf = nil
+	msg.Control.Len = 0
+	msg.Control.Buf = nil
 	wsaMsgPool.Put(msg)
 }
 
diff --git a/src/net/udpsock_test.go b/src/net/udpsock_test.go
index 7b4bf328e2..a79e9f83c1 100644
--- a/src/net/udpsock_test.go
+++ b/src/net/udpsock_test.go
@@ -8,6 +8,7 @@ import (
 	"errors"
 	"fmt"
 	"internal/asan"
+	"internal/race"
 	"internal/testenv"
 	"net/netip"
 	"os"
@@ -491,6 +492,12 @@ func TestAllocs(t *testing.T) {
 	case "plan9", "js", "wasip1":
 		// These implementations have not been optimized.
 		t.Skipf("skipping on %v", runtime.GOOS)
+	case "windows":
+		if race.Enabled {
+			// The Windows implementation make use of sync.Pool,
+			// which randomly drops cached items when race is enabled.
+			t.Skip("skipping test in race")
+		}
 	}
 	if !testableNetwork("udp4") {
 		t.Skipf("skipping: udp4 not available")