55 lines
2.7 KiB
Diff
55 lines
2.7 KiB
Diff
From: Lee Duncan <lduncan@suse.com>
|
||
Date: Thu, 11 Feb 2021 11:34:42 -0800
|
||
Subject: [PATCH] Fix test issue with gcc-11
|
||
MIME-Version: 1.0
|
||
Content-Type: text/plain; charset=UTF-8
|
||
Content-Transfer-Encoding: 8bit
|
||
|
||
Test 3.t exercises io_submit() with invalid addresses,
|
||
and one test attempted to pass in "-1" as an invalid
|
||
pointer. But gcc-11 realizes you cannot take an offset
|
||
from an invalid pointer, making it impossible to
|
||
compile or run this test:
|
||
|
||
> sh# make CC=gcc-11 partcheck
|
||
> make[1]: Entering directory '/alt/public_software/libaio/src'
|
||
> make[1]: Nothing to be done for 'all'.
|
||
> make[1]: Leaving directory '/alt/public_software/libaio/src'
|
||
> make[1]: Entering directory '/alt/public_software/libaio/harness'
|
||
> gcc-11 -Wall -Werror -I../src -g -O2 -DTEST_NAME=\"cases/2.t\" -o cases/2.p main.c ../src/libaio.a -lpthread
|
||
> gcc-11 -Wall -Werror -I../src -g -O2 -DTEST_NAME=\"cases/3.t\" -o cases/3.p main.c ../src/libaio.a -lpthread
|
||
> In file included from main.c:24:
|
||
> cases/3.t: In function ‘test_main’:
|
||
> cases/3.t:18:19: error: ‘attempt_io_submit’ accessing 8 bytes in a region of size 0 [-Werror=stringop-overflow=]
|
||
> 18 | status |= attempt_io_submit( io_ctx, 1, (void *)-1, -EFAULT);
|
||
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
|
||
> cases/3.t:18:19: note: referencing argument 3 of type ‘struct iocb **’
|
||
> In file included from cases/3.t:5,
|
||
> from main.c:24:
|
||
> cases/aio_setup.h:15:5: note: in a call to function ‘attempt_io_submit’
|
||
> 15 | int attempt_io_submit(io_context_t ctx, long nr, struct iocb *ios[], int expect)
|
||
> | ^~~~~~~~~~~~~~~~~
|
||
> cc1: all warnings being treated as errors
|
||
> make[1]: *** [Makefile:19: cases/3.p] Error 1
|
||
> make[1]: Leaving directory '/alt/public_software/libaio/harness'
|
||
> make: *** [Makefile:23: partcheck] Error 2
|
||
|
||
The fix is to remove this one test case, since the compiler now
|
||
ensures the running code will never see this case.
|
||
|
||
Signed-off-by: Lee Duncan <leeman.duncan@gmail.com>
|
||
---
|
||
harness/cases/3.t | 1 -
|
||
1 file changed, 1 deletion(-)
|
||
|
||
diff -Nurp a/harness/cases/3.t b/harness/cases/3.t
|
||
--- a/harness/cases/3.t 2018-10-22 20:59:34.000000000 +0100
|
||
+++ b/harness/cases/3.t 2022-02-26 09:46:51.969845084 +0000
|
||
@@ -15,7 +15,6 @@ int test_main(void)
|
||
status |= attempt_io_submit(BAD_CTX, 1, good_ios, -EINVAL);
|
||
status |= attempt_io_submit( io_ctx, 0, good_ios, 0);
|
||
status |= attempt_io_submit( io_ctx, 1, NULL, -EFAULT);
|
||
- status |= attempt_io_submit( io_ctx, 1, (void *)-1, -EFAULT);
|
||
status |= attempt_io_submit( io_ctx, 2, bad1_ios, -EFAULT);
|
||
status |= attempt_io_submit( io_ctx, 2, bad2_ios, -EFAULT);
|
||
status |= attempt_io_submit( io_ctx, -1, good_ios, -EINVAL);
|