From: Ian Lance Taylor <iant@golang.org>
Date: Wed, 6 Nov 2019 22:49:56 +0000 (-0800)
Subject: runtime: mark testSetPanicOnFault as go:nocheckptr
X-Git-Tag: go1.14beta1~372
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=6ce4384fd6621454a1d58a4ab46f92da8d8567ed;p=gostls13.git

runtime: mark testSetPanicOnFault as go:nocheckptr

The test deliberately constructs an invalid pointer, so don't check it.

Fixes #35379

Change-Id: Ifeff3484740786b0470de3a4d2d4103d91e06f5d
Reviewed-on: https://go-review.googlesource.com/c/go/+/205717
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
---

diff --git a/src/runtime/runtime_test.go b/src/runtime/runtime_test.go
index 26ae77456a..e5d2d97d05 100644
--- a/src/runtime/runtime_test.go
+++ b/src/runtime/runtime_test.go
@@ -192,6 +192,10 @@ func TestSetPanicOnFault(t *testing.T) {
 	}
 }
 
+// testSetPanicOnFault tests one potentially faulting address.
+// It deliberately constructs and uses an invalid pointer,
+// so mark it as nocheckptr.
+//go:nocheckptr
 func testSetPanicOnFault(t *testing.T, addr uintptr, nfault *int) {
 	if GOOS == "js" {
 		t.Skip("js does not support catching faults")