diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2020-11-27 20:45:15 +0100 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2020-11-27 20:31:33 +0000 |
commit | 91f77ca2f8590ed2051ee9a62d52676cf1bff98d (patch) | |
tree | a26d114ead48992a41d3c6188cdd7b1210a2da98 /src/runtime/syscall_windows_test.go | |
parent | 926994fd7cf65b2703552686965fb05569699897 (diff) | |
download | go-91f77ca2f8590ed2051ee9a62d52676cf1bff98d.tar.gz go-91f77ca2f8590ed2051ee9a62d52676cf1bff98d.zip |
runtime: return 0 from C function in test
This function's prototype includes a return value, so return a value.
Otherwise clang gets upset:
--- FAIL: TestDLLPreloadMitigation (1.40s)
syscall_windows_test.go:986: failed to build dll: exit status 1 - nojack.c:7:1: error: non-void function does not return a value [-Werror,-Wreturn-type]
}
^
1 error generated.
Fixes #42860.
Change-Id: I65b8eb9ccb502692c5b65bd34829f331cd86eef0
Reviewed-on: https://go-review.googlesource.com/c/go/+/273726
Trust: Jason A. Donenfeld <Jason@zx2c4.com>
Trust: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/runtime/syscall_windows_test.go')
-rw-r--r-- | src/runtime/syscall_windows_test.go | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/runtime/syscall_windows_test.go b/src/runtime/syscall_windows_test.go index 3827c6ed83..a20573eb6a 100644 --- a/src/runtime/syscall_windows_test.go +++ b/src/runtime/syscall_windows_test.go @@ -969,8 +969,9 @@ func TestDLLPreloadMitigation(t *testing.T) { #include <stdint.h> #include <windows.h> -uintptr_t cfunc() { +uintptr_t cfunc(void) { SetLastError(123); + return 0; } ` srcname := "nojack.c" |