From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2201) id 9860F38654B0; Fri, 13 Jan 2023 17:05:00 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 9860F38654B0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1673629500; bh=riE5BmIxW0n+uw/ouYe6kifIbZINcFxK47RlBv10OEs=; h=From:To:Subject:Date:From; b=iXUd0FWDRwGPfqHL9Uv+Uw2awpJZvY1xss1/lSNbcVQlxu+nDU/q+3NbFvB5jtWmv tWJEXZZ3hU3pS74MYz+RhYINH0T4woSL67Nkocy3HeD+9qHBMuDPXiPvMDp+b7Z9LO nQxzxg3+IK6ZO+JptraUwCPUe/8FS4SMWYMtzp3U= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Jon Turney To: cygwin-cvs@sourceware.org Subject: [newlib-cygwin] Cygwin: testsuite: Fix compilation warnings X-Act-Checkin: newlib-cygwin X-Git-Author: Jon Turney X-Git-Refname: refs/heads/master X-Git-Oldrev: daeeaa97b35c0f143d6aa9d97d94e23e375337e8 X-Git-Newrev: 01a0e3e4917dda546f373f2d806f6e6c348faa82 Message-Id: <20230113170500.9860F38654B0@sourceware.org> Date: Fri, 13 Jan 2023 17:05:00 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=3Dnewlib-cygwin.git;h=3D01a0e3e4917= dda546f373f2d806f6e6c348faa82 commit 01a0e3e4917dda546f373f2d806f6e6c348faa82 Author: Jon Turney Date: Tue Aug 30 14:22:55 2022 +0100 Cygwin: testsuite: Fix compilation warnings =20 Fix the warnings the previous change surfaces. =20 See ltp commit d5c2112f for mmap fixes. Diff: --- winsup/testsuite/libltp/include/usctest.h | 2 +- winsup/testsuite/winsup.api/ltp/execv01.c | 2 +- winsup/testsuite/winsup.api/ltp/execve01.c | 2 +- winsup/testsuite/winsup.api/ltp/execvp01.c | 2 +- winsup/testsuite/winsup.api/ltp/mmap02.c | 10 ++++------ winsup/testsuite/winsup.api/ltp/mmap03.c | 10 ++++------ winsup/testsuite/winsup.api/ltp/mmap04.c | 10 ++++------ winsup/testsuite/winsup.api/ltp/mmap05.c | 10 +++++----- winsup/testsuite/winsup.api/ltp/mmap06.c | 8 +++++--- winsup/testsuite/winsup.api/ltp/mmap07.c | 8 +++++--- winsup/testsuite/winsup.api/ltp/mmap08.c | 8 +++++--- winsup/testsuite/winsup.api/mmaptest03.c | 2 +- winsup/testsuite/winsup.api/systemcall.c | 2 +- winsup/testsuite/winsup.api/user_malloc.c | 4 ++-- 14 files changed, 40 insertions(+), 40 deletions(-) diff --git a/winsup/testsuite/libltp/include/usctest.h b/winsup/testsuite/l= ibltp/include/usctest.h index fef349d04..637635a25 100644 --- a/winsup/testsuite/libltp/include/usctest.h +++ b/winsup/testsuite/libltp/include/usctest.h @@ -210,7 +210,7 @@ extern void STD_opts_help(); * SCALL =3D system call and parameters to execute * ***********************************************************************/ -#define TEST(SCALL) TEST_RETURN =3D SCALL; TEST_ERRNO=3Derrno; +#define TEST(SCALL) TEST_RETURN =3D (long) SCALL; TEST_ERRNO=3Derrno; =20 /*********************************************************************** * TEST_VOID: calls a system call diff --git a/winsup/testsuite/winsup.api/ltp/execv01.c b/winsup/testsuite/w= insup.api/ltp/execv01.c index f59f29702..dca62c523 100644 --- a/winsup/testsuite/winsup.api/ltp/execv01.c +++ b/winsup/testsuite/winsup.api/ltp/execv01.c @@ -130,7 +130,7 @@ int exp_enos[]=3D{0, 0}; /* Zero terminated list of ex= pected errnos */ =20 int pid; /* process id from fork */ int status; /* status returned from waitpid */ -const char * const args[2]=3D{"/usr/bin/test", 0}; /* argument list for ex= ecv call */ +char * const args[2]=3D{"/usr/bin/test", 0}; /* argument list for execv ca= ll */ =20 int main(int ac, char **av) diff --git a/winsup/testsuite/winsup.api/ltp/execve01.c b/winsup/testsuite/= winsup.api/ltp/execve01.c index 2584bdf05..eb5073d31 100644 --- a/winsup/testsuite/winsup.api/ltp/execve01.c +++ b/winsup/testsuite/winsup.api/ltp/execve01.c @@ -133,7 +133,7 @@ int exp_enos[]=3D{0, 0}; /* Zero terminated list of ex= pected errnos */ =20 int pid; /* process id from fork */ int status; /* status returned from waitpid */ -const char *const args[2]=3D{"/usr/bin/test", 0}; /* argument list for exe= cve call */ +char *const args[2]=3D{"/usr/bin/test", 0}; /* argument list for execve ca= ll */ extern char **environ; /* pointer to this processes env, to pass along */ =20 int diff --git a/winsup/testsuite/winsup.api/ltp/execvp01.c b/winsup/testsuite/= winsup.api/ltp/execvp01.c index 8a1726a21..1473ccf66 100644 --- a/winsup/testsuite/winsup.api/ltp/execvp01.c +++ b/winsup/testsuite/winsup.api/ltp/execvp01.c @@ -133,7 +133,7 @@ int exp_enos[]=3D{0, 0}; /* Zero terminated list of ex= pected errnos */ =20 int pid; /* process id from fork */ int status; /* status returned from waitpid */ -const char *const args[2]=3D{"/usr/bin/test", 0}; /* argument list for exe= cvp call */ +char *const args[2]=3D{"/usr/bin/test", 0}; /* argument list for execvp ca= ll */ =20 int main(int ac, char **av) diff --git a/winsup/testsuite/winsup.api/ltp/mmap02.c b/winsup/testsuite/wi= nsup.api/ltp/mmap02.c index b96bdb452..ca9f4d956 100644 --- a/winsup/testsuite/winsup.api/ltp/mmap02.c +++ b/winsup/testsuite/winsup.api/ltp/mmap02.c @@ -118,11 +118,12 @@ main(int ac, char **av) * Call mmap to map the temporary file 'TEMPFILE' * with read access. */ - TEST(mmap(0, page_sz, PROT_READ, - MAP_FILE|MAP_SHARED, fildes, 0)); + errno =3D 0; + addr =3D mmap(0, page_sz, PROT_READ, + MAP_FILE|MAP_SHARED, fildes, 0); =20 /* Check for the return value of mmap() */ - if (TEST_RETURN =3D=3D (int)MAP_FAILED) { + if (addr =3D=3D MAP_FAILED) { tst_resm(TFAIL, "mmap() Failed on %s, errno=3D%d : %s", TEMPFILE, errno, strerror(errno)); continue; @@ -132,9 +133,6 @@ main(int ac, char **av) * executed without (-f) option. */ if (STD_FUNCTIONAL_TEST) { - /* Get the mmap return value */ - addr =3D (char *)TEST_RETURN; - /* * Read the file contents into the dummy * string. diff --git a/winsup/testsuite/winsup.api/ltp/mmap03.c b/winsup/testsuite/wi= nsup.api/ltp/mmap03.c index fba512c28..5de5435d3 100644 --- a/winsup/testsuite/winsup.api/ltp/mmap03.c +++ b/winsup/testsuite/winsup.api/ltp/mmap03.c @@ -121,11 +121,12 @@ main(int ac, char **av) * Call mmap to map the temporary file 'TEMPFILE' * with execute access. */ - TEST(mmap(0, page_sz, PROT_EXEC, - MAP_FILE|MAP_SHARED, fildes, 0)); + errno =3D 0; + addr =3D mmap(0, page_sz, PROT_EXEC, + MAP_FILE|MAP_SHARED, fildes, 0); =20 /* Check for the return value of mmap() */ - if (TEST_RETURN =3D=3D (int)MAP_FAILED) { + if (addr =3D=3D MAP_FAILED) { tst_resm(TFAIL, "mmap() Failed on %s, errno=3D%d : %s", TEMPFILE, errno, strerror(errno)); continue; @@ -135,9 +136,6 @@ main(int ac, char **av) * executed without (-f) option. */ if (STD_FUNCTIONAL_TEST) { - /* Get the mmap return value */ - addr =3D (char *)TEST_RETURN; - /* * Read the file contents into the dummy * variable. diff --git a/winsup/testsuite/winsup.api/ltp/mmap04.c b/winsup/testsuite/wi= nsup.api/ltp/mmap04.c index dbe25aefd..e69d15a97 100644 --- a/winsup/testsuite/winsup.api/ltp/mmap04.c +++ b/winsup/testsuite/winsup.api/ltp/mmap04.c @@ -121,11 +121,12 @@ main(int ac, char **av) * Call mmap to map the temporary file 'TEMPFILE' * with read and execute access. */ - TEST(mmap(0, page_sz, PROT_READ|PROT_EXEC, - MAP_FILE|MAP_SHARED, fildes, 0)); + errno =3D 0; + addr =3D mmap(0, page_sz, PROT_READ|PROT_EXEC, + MAP_FILE|MAP_SHARED, fildes, 0); =20 /* Check for the return value of mmap() */ - if (TEST_RETURN =3D=3D (int)MAP_FAILED) { + if (addr =3D=3D MAP_FAILED) { tst_resm(TFAIL, "mmap() Failed on %s, errno=3D%d : %s", TEMPFILE, errno, strerror(errno)); continue; @@ -136,9 +137,6 @@ main(int ac, char **av) * executed without (-f) option. */ if (STD_FUNCTIONAL_TEST) { - /* Get the mmap return value */ - addr =3D (char *)TEST_RETURN; - /* * Read the file contents into the dummy * variable. diff --git a/winsup/testsuite/winsup.api/ltp/mmap05.c b/winsup/testsuite/wi= nsup.api/ltp/mmap05.c index 6e75ee222..600cd761e 100644 --- a/winsup/testsuite/winsup.api/ltp/mmap05.c +++ b/winsup/testsuite/winsup.api/ltp/mmap05.c @@ -125,12 +125,12 @@ main(int ac, char **av) * Call mmap to map the temporary file 'TEMPFILE' * with no access. */ - =09 - TEST(mmap(0, page_sz, PROT_NONE, - MAP_FILE|MAP_SHARED, fildes, 0)); + errno =3D 0; + addr =3D mmap(0, page_sz, PROT_NONE, + MAP_FILE|MAP_SHARED, fildes, 0); =20 /* Check for the return value of mmap() */ - if (TEST_RETURN =3D=3D (int)MAP_FAILED) { + if (addr =3D=3D MAP_FAILED) { tst_resm(TFAIL, "mmap() Failed on %s, errno=3D%d : %s", TEMPFILE, errno, strerror(errno)); continue; @@ -264,7 +264,7 @@ setup() * is not accessible. */ void -sig_handler(sig) +sig_handler(int sig) { if (sig =3D=3D SIGSEGV) { /* set the global variable and jump back */ diff --git a/winsup/testsuite/winsup.api/ltp/mmap06.c b/winsup/testsuite/wi= nsup.api/ltp/mmap06.c index c099f8c33..d089f900b 100644 --- a/winsup/testsuite/winsup.api/ltp/mmap06.c +++ b/winsup/testsuite/winsup.api/ltp/mmap06.c @@ -121,11 +121,13 @@ main(int ac, char **av) * Call mmap to map the temporary file 'TEMPFILE' * with read access. */ - TEST(mmap(0, page_sz, PROT_READ, - MAP_FILE|MAP_SHARED, fildes, 0)); + errno =3D 0; + addr =3D mmap(0, page_sz, PROT_READ, + MAP_FILE|MAP_SHARED, fildes, 0); + TEST_ERRNO =3D errno; =20 /* Check for the return value of mmap() */ - if (TEST_RETURN !=3D (int)MAP_FAILED) { + if (addr !=3D MAP_FAILED) { tst_resm(TFAIL, "mmap() returned invalid value, " "expected: -1"); /* Unmap the mapped memory */ diff --git a/winsup/testsuite/winsup.api/ltp/mmap07.c b/winsup/testsuite/wi= nsup.api/ltp/mmap07.c index 6e3bb5112..4be6129e6 100644 --- a/winsup/testsuite/winsup.api/ltp/mmap07.c +++ b/winsup/testsuite/winsup.api/ltp/mmap07.c @@ -122,11 +122,13 @@ main(int ac, char **av) * Call mmap to map the temporary file 'TEMPFILE' * with write access. */ - TEST(mmap(0, page_sz, PROT_WRITE, - MAP_FILE|MAP_PRIVATE, fildes, 0)); + errno =3D 0; + addr =3D mmap(0, page_sz, PROT_WRITE, + MAP_FILE|MAP_PRIVATE, fildes, 0); + TEST_ERRNO =3D errno; =20 /* Check for the return value of mmap() */ - if (TEST_RETURN !=3D (int)MAP_FAILED) { + if (addr !=3D MAP_FAILED) { tst_resm(TFAIL, "mmap() returned invalid value, " "expected: -1"); /* Unmap the mapped memory */ diff --git a/winsup/testsuite/winsup.api/ltp/mmap08.c b/winsup/testsuite/wi= nsup.api/ltp/mmap08.c index 543c5397b..49b6471e5 100644 --- a/winsup/testsuite/winsup.api/ltp/mmap08.c +++ b/winsup/testsuite/winsup.api/ltp/mmap08.c @@ -117,11 +117,13 @@ main(int ac, char **av) * Call mmap to map the temporary file 'TEMPFILE' * which is already closed. so, fildes is not valid. */ - TEST(mmap(0, page_sz, PROT_WRITE, - MAP_FILE|MAP_SHARED, fildes, 0)); + errno =3D 0; + addr =3D mmap(0, page_sz, PROT_WRITE, + MAP_FILE|MAP_SHARED, fildes, 0); + TEST_ERRNO =3D errno; =20 /* Check for the return value of mmap() */ - if (TEST_RETURN !=3D (int)MAP_FAILED) { + if (addr !=3D MAP_FAILED) { tst_resm(TFAIL, "mmap() returned invalid value, " "expected: -1"); /* Unmap the mapped memory */ diff --git a/winsup/testsuite/winsup.api/mmaptest03.c b/winsup/testsuite/wi= nsup.api/mmaptest03.c index 8046f0bf8..e28e0f251 100644 --- a/winsup/testsuite/winsup.api/mmaptest03.c +++ b/winsup/testsuite/winsup.api/mmaptest03.c @@ -145,7 +145,7 @@ main(int argc, char **argv) unlink ("y.txt"); if (!WIFEXITED (status) || WEXITSTATUS (status)) { - printf ("forked process exited with status %p\n", (char *) status); + printf ("forked process exited with status %x\n", status); return 1; } } diff --git a/winsup/testsuite/winsup.api/systemcall.c b/winsup/testsuite/wi= nsup.api/systemcall.c index 91dd748c0..d10c9825c 100644 --- a/winsup/testsuite/winsup.api/systemcall.c +++ b/winsup/testsuite/winsup.api/systemcall.c @@ -61,7 +61,7 @@ main (int argc, char **argv) } if (n !=3D 0) { - fprintf (stderr, "system() call returned %p\n", (void *) n); + fprintf (stderr, "system() call returned %x\n", n); exit (1); } exit (0); diff --git a/winsup/testsuite/winsup.api/user_malloc.c b/winsup/testsuite/w= insup.api/user_malloc.c index 8685f86ab..e2b1e0a92 100644 --- a/winsup/testsuite/winsup.api/user_malloc.c +++ b/winsup/testsuite/winsup.api/user_malloc.c @@ -133,11 +133,11 @@ ull * current =3D buffer; =20 static int is_valid (void * ptr) { - unsigned int iptr =3D (unsigned int) ptr; + uintptr_t iptr =3D (uintptr_t) ptr; ull * ullptr =3D (ull *) ptr; =20 iptr =3D (iptr / sizeof(ull)) * sizeof(ull); - if (iptr !=3D (int) ptr) + if (iptr !=3D (uintptr_t) ptr) return 0; if (--ullptr < buffer || ullptr[0] > SIZE || ullptr + ullptr[0] > curr= ent) return 0;