From a2360d0742f01e40bf4fb1714de4503f8a82aa3f Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Tue, 15 Sep 2020 08:26:45 +0200 Subject: executor: fix definition of __NR_io_uring_setup MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sone syzbot instances broke with: : In function ‘syz_io_uring_setup’: :476:33: error: ‘__NR_io_uring_setup’ undeclared (first use in this function) :476:33: note: each undeclared identifier is reported only once for each function it appears in pkg/csource resolves #ifdef's at generation time. --- executor/common_linux.h | 12 +++++------- executor/style_test.go | 18 ++++++++++++++++++ pkg/csource/generated.go | 10 ++++------ 3 files changed, 27 insertions(+), 13 deletions(-) diff --git a/executor/common_linux.h b/executor/common_linux.h index a9ca4ac27..2c45387d8 100644 --- a/executor/common_linux.h +++ b/executor/common_linux.h @@ -1467,13 +1467,11 @@ struct io_uring_params { #include #include -#ifndef __NR_io_uring_setup -#ifdef __alpha__ -#define __NR_io_uring_setup 535 -#else // !__alpha__ -#define __NR_io_uring_setup 425 +#if GOARCH_mips64le +#define sys_io_uring_setup 5425 +#else +#define sys_io_uring_setup 425 #endif -#endif // __NR_io_uring_setup // Wrapper for io_uring_setup and the subsequent mmap calls that map the ring and the sqes static long syz_io_uring_setup(volatile long a0, volatile long a1, volatile long a2, volatile long a3, volatile long a4, volatile long a5) @@ -1489,7 +1487,7 @@ static long syz_io_uring_setup(volatile long a0, volatile long a1, volatile long void** ring_ptr_out = (void**)a4; void** sqes_ptr_out = (void**)a5; - uint32 fd_io_uring = syscall(__NR_io_uring_setup, entries, setup_params); + uint32 fd_io_uring = syscall(sys_io_uring_setup, entries, setup_params); // Compute the ring sizes uint32 sq_ring_sz = setup_params->sq_off.array + setup_params->sq_entries * sizeof(uint32); diff --git a/executor/style_test.go b/executor/style_test.go index 488752626..f6f5856f3 100644 --- a/executor/style_test.go +++ b/executor/style_test.go @@ -43,6 +43,24 @@ if (foo) `/* C++ comment */`, }, }, + { + pattern: `#define __NR_`, + message: "Don't define syscall __NR_foo constants.\n" + + "These should be guarded by #ifndef __NR_foo, but this is dependent on the host " + + "and may break on other machines (after pkg/csource processing).\n" + + "Define sys_foo constants instead.", + tests: []string{ + ` +#ifndef __NR_io_uring_setup +#ifdef __alpha__ +#define __NR_io_uring_setup 535 +#else // !__alpha__ +#define __NR_io_uring_setup 425 +#endif +#endif // __NR_io_uring_setup +`, + }, + }, { pattern: `//[^\s]`, suppression: `https?://`, diff --git a/pkg/csource/generated.go b/pkg/csource/generated.go index 65abca7e7..6d9009907 100644 --- a/pkg/csource/generated.go +++ b/pkg/csource/generated.go @@ -3589,12 +3589,10 @@ struct io_uring_params { #include #include -#ifndef __NR_io_uring_setup -#ifdef __alpha__ -#define __NR_io_uring_setup 535 +#if GOARCH_mips64le +#define sys_io_uring_setup 5425 #else -#define __NR_io_uring_setup 425 -#endif +#define sys_io_uring_setup 425 #endif static long syz_io_uring_setup(volatile long a0, volatile long a1, volatile long a2, volatile long a3, volatile long a4, volatile long a5) { @@ -3605,7 +3603,7 @@ static long syz_io_uring_setup(volatile long a0, volatile long a1, volatile long void** ring_ptr_out = (void**)a4; void** sqes_ptr_out = (void**)a5; - uint32 fd_io_uring = syscall(__NR_io_uring_setup, entries, setup_params); + uint32 fd_io_uring = syscall(sys_io_uring_setup, entries, setup_params); uint32 sq_ring_sz = setup_params->sq_off.array + setup_params->sq_entries * sizeof(uint32); uint32 cq_ring_sz = setup_params->cq_off.cqes + setup_params->cq_entries * SIZEOF_IO_URING_CQE; uint32 ring_sz = sq_ring_sz > cq_ring_sz ? sq_ring_sz : cq_ring_sz; -- cgit mrf-deployment