Commit dd40f44e authored Oct 21, 2021 by Shuah Khan

selftests: x86: fix [-Wstringop-overread] warn in test_process_vm_readv()



Fix the following [-Wstringop-overread] by passing in the variable
instead of the value.

test_vsyscall.c: In function ‘test_process_vm_readv’:
test_vsyscall.c:500:22: warning: ‘__builtin_memcmp_eq’ specified bound 4096 exceeds source size 0 [-Wstringop-overread]
  500 |                 if (!memcmp(buf, (const void *)0xffffffffff600000, 4096)) {
      |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>

parent c3867ab5

tools/testing/selftests/x86/test_vsyscall.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -497,7 +497,7 @@ static int test_process_vm_readv(void)
		}

		if (vsyscall_map_r) {
		if (!memcmp(buf, (const void *)0xffffffffff600000, 4096)) {
		if (!memcmp(buf, remote.iov_base, sizeof(buf))) {
		printf("[OK]\tIt worked and read correct data\n");
		} else {
		printf("[FAIL]\tIt worked but returned incorrect data\n");