#9 master: test suite compile
Opened 2 years ago by kloczek. Modified 2 years ago

+ /usr/bin/make -O -j1 V=1 VERBOSE=1 check
make[1]: Entering directory '/home/tkloczko/rpmbuild/BUILD/libaio-0.3.112/harness'
gcc -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fstack-protector-strong -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection  -fdata-sections -ffunction-sections -Os -Wall -Werror -I../src -g -O2 -DTEST_NAME=\"cases/2.t\" -o cases/2.p main.c ../src/libaio.a -lpthread
gcc -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fstack-protector-strong -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection  -fdata-sections -ffunction-sections -Os -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

I found this same issue with gcc-11.

I will submit the following patch:

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 --git a/harness/cases/3.t b/harness/cases/3.t
index 7773d80f0665..ba669a6494ce 100644
--- a/harness/cases/3.t
+++ b/harness/cases/3.t
@@ -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);
-- 
2.30.0

Login to comment on this ticket.

Metadata