From: Khaled Yakdan Date: Sat, 25 Jun 2022 05:17:45 +0000 (+0000) Subject: runtime: mark string comparison hooks as no split X-Git-Tag: go1.19rc1~62 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=1821639b57a643f9c3f6d3b3d329d2283d5d7a57;p=gostls13.git runtime: mark string comparison hooks as no split These functions can be inserted by the compiler into the code to be instrumented. This may result in these functions having callers that are nosplit. That is why they must be nosplit. This is a followup for CL 410034 in order to fix #53190. Change-Id: I03746208a2a302a581a1eaad6c9d0672bb1e949a GitHub-Last-Rev: 6506d86f221d745de083fad862bba7ba04a80455 GitHub-Pull-Request: golang/go#53544 Reviewed-on: https://go-review.googlesource.com/c/go/+/413978 TryBot-Result: Gopher Robot Reviewed-by: Keith Randall Reviewed-by: Keith Randall Reviewed-by: Dmitri Shuralyov Run-TryBot: Dmitri Shuralyov --- diff --git a/src/runtime/libfuzzer.go b/src/runtime/libfuzzer.go index 02dcc18e7a..6bfaef823b 100644 --- a/src/runtime/libfuzzer.go +++ b/src/runtime/libfuzzer.go @@ -92,6 +92,8 @@ func init() { // 4. result: an integer representing the comparison result. 0 indicates // equality (comparison will ignored by libfuzzer), non-zero indicates a // difference (comparison will be taken into consideration). +// +//go:nosplit func libfuzzerHookStrCmp(s1, s2 string, fakePC int) { if s1 != s2 { libfuzzerCall4(&__sanitizer_weak_hook_strcmp, uintptr(fakePC), cstring(s1), cstring(s2), uintptr(1)) @@ -102,6 +104,8 @@ func libfuzzerHookStrCmp(s1, s2 string, fakePC int) { // This function has now the same implementation as libfuzzerHookStrCmp because we lack better checks // for case-insensitive string equality in the runtime package. +// +//go:nosplit func libfuzzerHookEqualFold(s1, s2 string, fakePC int) { if s1 != s2 { libfuzzerCall4(&__sanitizer_weak_hook_strcmp, uintptr(fakePC), cstring(s1), cstring(s2), uintptr(1))