From 6ce4384fd6621454a1d58a4ab46f92da8d8567ed Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Wed, 6 Nov 2019 14:49:56 -0800 Subject: [PATCH] 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 --- src/runtime/runtime_test.go | 4 ++++ 1 file changed, 4 insertions(+) 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") -- 2.50.0