From: Dmitry V. Levin <ldv@altlinux.org>
Date: Sun, 16 Sep 2018 09:48:16 +0000 (+0000)
Subject: tests: workaround clang -Wunneeded-internal-declaration bug
X-Git-Tag: v4.25~17
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=d7cbc89824a1b2a1ce574309ee36114374fbacdb;p=strace

tests: workaround clang -Wunneeded-internal-declaration bug

* tests/execve.c (main): Explicitly mark q_envp variable as used
to workaround clang -Wunneeded-internal-declaration bug.
* tests/execveat.c (main): Likewise.
* tests/nsyscalls.c (main): Explicitly mark syscallent variable as used
to workaround clang -Wunneeded-internal-declaration bug.

Closes: https://github.com/strace/strace/issues/78
---

diff --git a/tests/execve.c b/tests/execve.c
index c1dedfcc..ce291fe1 100644
--- a/tests/execve.c
+++ b/tests/execve.c
@@ -78,6 +78,7 @@ main(void)
 
 	tail_argv[ARRAY_SIZE(q_argv)] = NULL;
 	tail_envp[ARRAY_SIZE(q_envp)] = NULL;
+	(void) q_envp;	/* workaround for clang bug #33068 */
 
 	execve(FILENAME, tail_argv, tail_envp);
 	printf("execve(\"%s\", [\"%s\", \"%s\", \"%s\"]"
diff --git a/tests/execveat.c b/tests/execveat.c
index 853e26c4..f4c88d1b 100644
--- a/tests/execveat.c
+++ b/tests/execveat.c
@@ -83,6 +83,7 @@ main(void)
 
 	tail_argv[ARRAY_SIZE(q_argv)] = NULL;
 	tail_envp[ARRAY_SIZE(q_envp)] = NULL;
+	(void) q_envp;	/* workaround for clang bug #33068 */
 
 	syscall(__NR_execveat, -100, FILENAME, tail_argv, tail_envp, 0x1100);
 	printf("execveat(AT_FDCWD, \"%s\", [\"%s\", \"%s\", \"%s\"]"
diff --git a/tests/nsyscalls.c b/tests/nsyscalls.c
index e2ebbf99..e3544232 100644
--- a/tests/nsyscalls.c
+++ b/tests/nsyscalls.c
@@ -119,6 +119,7 @@ main(int argc, char *argv[])
 #endif
 
 	test_syscall(ARRAY_SIZE(syscallent));
+	(void) syscallent;	/* workaround for clang bug #33068 */
 
 #ifdef SYS_socket_subcall
 	test_syscall(SYS_socket_subcall + 1);