]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: mark testSetPanicOnFault as go:nocheckptr
authorIan Lance Taylor <iant@golang.org>
Wed, 6 Nov 2019 22:49:56 +0000 (14:49 -0800)
committerIan Lance Taylor <iant@golang.org>
Wed, 6 Nov 2019 23:48:45 +0000 (23:48 +0000)
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>
src/runtime/runtime_test.go

index 26ae77456ae6d6148caa0b307cd5c832007b6f8f..e5d2d97d05f184a14e0cd9ab96ff9fc93c9f9d2d 100644 (file)
@@ -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")