aboutsummaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorKeith Randall <khr@golang.org>2025-08-07 15:30:54 -0700
committerGopher Robot <gobot@golang.org>2025-08-08 06:51:37 -0700
commitb3388569a187ea6be48caa41265f2b4dbc2fdfd3 (patch)
treeb4c1a95ca94f80fc817637399c36557bcc31b073 /test
parentd83b16fcb8de765f25cabbea63284406ea6dd091 (diff)
downloadgo-b3388569a187ea6be48caa41265f2b4dbc2fdfd3.tar.xz
reflect: handle zero-sized fields of directly-stored structures correctly
type W struct { E struct{} X *byte } type W is a "direct" type. That is, it is a pointer-ish type that can be stored directly as the second word of an interface. But if we ask reflect for W's first field, that value must *not* be direct, as zero-sized things cannot be stored directly. This was a problem introduced in CL 681937. Before that, types like W were not eligible for directness. Fixes #74935 Change-Id: Idefb55c23eaa59153009f863bad611593981e5cb Reviewed-on: https://go-review.googlesource.com/c/go/+/694195 Auto-Submit: Keith Randall <khr@golang.org> Reviewed-by: Keith Randall <khr@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Cuong Manh Le <cuong.manhle.vn@gmail.com> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Diffstat (limited to 'test')
-rw-r--r--test/fixedbugs/issue74935.go19
1 files changed, 19 insertions, 0 deletions
diff --git a/test/fixedbugs/issue74935.go b/test/fixedbugs/issue74935.go
new file mode 100644
index 0000000000..1f6f718b02
--- /dev/null
+++ b/test/fixedbugs/issue74935.go
@@ -0,0 +1,19 @@
+// run
+
+// Copyright 2025 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package main
+
+import "reflect"
+
+type W struct {
+ E struct{}
+ X *byte
+}
+
+func main() {
+ w := reflect.ValueOf(W{})
+ _ = w.Field(0).Interface()
+}