public inbox for cygwin-cvs@sourceware.org
help / color / mirror / Atom feed
* [newlib-cygwin] Cygwin: testsuite: Fix compilation warnings
@ 2023-01-13 17:05 Jon Turney
0 siblings, 0 replies; only message in thread
From: Jon Turney @ 2023-01-13 17:05 UTC (permalink / raw)
To: cygwin-cvs
https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=01a0e3e4917dda546f373f2d806f6e6c348faa82
commit 01a0e3e4917dda546f373f2d806f6e6c348faa82
Author: Jon Turney <jon.turney@dronecode.org.uk>
Date: Tue Aug 30 14:22:55 2022 +0100
Cygwin: testsuite: Fix compilation warnings
Fix the warnings the previous change surfaces.
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/libltp/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 = system call and parameters to execute
*
***********************************************************************/
-#define TEST(SCALL) TEST_RETURN = SCALL; TEST_ERRNO=errno;
+#define TEST(SCALL) TEST_RETURN = (long) SCALL; TEST_ERRNO=errno;
/***********************************************************************
* TEST_VOID: calls a system call
diff --git a/winsup/testsuite/winsup.api/ltp/execv01.c b/winsup/testsuite/winsup.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[]={0, 0}; /* Zero terminated list of expected errnos */
int pid; /* process id from fork */
int status; /* status returned from waitpid */
-const char * const args[2]={"/usr/bin/test", 0}; /* argument list for execv call */
+char * const args[2]={"/usr/bin/test", 0}; /* argument list for execv call */
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[]={0, 0}; /* Zero terminated list of expected errnos */
int pid; /* process id from fork */
int status; /* status returned from waitpid */
-const char *const args[2]={"/usr/bin/test", 0}; /* argument list for execve call */
+char *const args[2]={"/usr/bin/test", 0}; /* argument list for execve call */
extern char **environ; /* pointer to this processes env, to pass along */
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[]={0, 0}; /* Zero terminated list of expected errnos */
int pid; /* process id from fork */
int status; /* status returned from waitpid */
-const char *const args[2]={"/usr/bin/test", 0}; /* argument list for execvp call */
+char *const args[2]={"/usr/bin/test", 0}; /* argument list for execvp call */
int
main(int ac, char **av)
diff --git a/winsup/testsuite/winsup.api/ltp/mmap02.c b/winsup/testsuite/winsup.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 = 0;
+ addr = mmap(0, page_sz, PROT_READ,
+ MAP_FILE|MAP_SHARED, fildes, 0);
/* Check for the return value of mmap() */
- if (TEST_RETURN == (int)MAP_FAILED) {
+ if (addr == MAP_FAILED) {
tst_resm(TFAIL, "mmap() Failed on %s, errno=%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 = (char *)TEST_RETURN;
-
/*
* Read the file contents into the dummy
* string.
diff --git a/winsup/testsuite/winsup.api/ltp/mmap03.c b/winsup/testsuite/winsup.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 = 0;
+ addr = mmap(0, page_sz, PROT_EXEC,
+ MAP_FILE|MAP_SHARED, fildes, 0);
/* Check for the return value of mmap() */
- if (TEST_RETURN == (int)MAP_FAILED) {
+ if (addr == MAP_FAILED) {
tst_resm(TFAIL, "mmap() Failed on %s, errno=%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 = (char *)TEST_RETURN;
-
/*
* Read the file contents into the dummy
* variable.
diff --git a/winsup/testsuite/winsup.api/ltp/mmap04.c b/winsup/testsuite/winsup.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 = 0;
+ addr = mmap(0, page_sz, PROT_READ|PROT_EXEC,
+ MAP_FILE|MAP_SHARED, fildes, 0);
/* Check for the return value of mmap() */
- if (TEST_RETURN == (int)MAP_FAILED) {
+ if (addr == MAP_FAILED) {
tst_resm(TFAIL, "mmap() Failed on %s, errno=%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 = (char *)TEST_RETURN;
-
/*
* Read the file contents into the dummy
* variable.
diff --git a/winsup/testsuite/winsup.api/ltp/mmap05.c b/winsup/testsuite/winsup.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.
*/
-
- TEST(mmap(0, page_sz, PROT_NONE,
- MAP_FILE|MAP_SHARED, fildes, 0));
+ errno = 0;
+ addr = mmap(0, page_sz, PROT_NONE,
+ MAP_FILE|MAP_SHARED, fildes, 0);
/* Check for the return value of mmap() */
- if (TEST_RETURN == (int)MAP_FAILED) {
+ if (addr == MAP_FAILED) {
tst_resm(TFAIL, "mmap() Failed on %s, errno=%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 == SIGSEGV) {
/* set the global variable and jump back */
diff --git a/winsup/testsuite/winsup.api/ltp/mmap06.c b/winsup/testsuite/winsup.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 = 0;
+ addr = mmap(0, page_sz, PROT_READ,
+ MAP_FILE|MAP_SHARED, fildes, 0);
+ TEST_ERRNO = errno;
/* Check for the return value of mmap() */
- if (TEST_RETURN != (int)MAP_FAILED) {
+ if (addr != 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/winsup.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 = 0;
+ addr = mmap(0, page_sz, PROT_WRITE,
+ MAP_FILE|MAP_PRIVATE, fildes, 0);
+ TEST_ERRNO = errno;
/* Check for the return value of mmap() */
- if (TEST_RETURN != (int)MAP_FAILED) {
+ if (addr != 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/winsup.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 = 0;
+ addr = mmap(0, page_sz, PROT_WRITE,
+ MAP_FILE|MAP_SHARED, fildes, 0);
+ TEST_ERRNO = errno;
/* Check for the return value of mmap() */
- if (TEST_RETURN != (int)MAP_FAILED) {
+ if (addr != 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/winsup.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/winsup.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 != 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/winsup.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 = buffer;
static int is_valid (void * ptr)
{
- unsigned int iptr = (unsigned int) ptr;
+ uintptr_t iptr = (uintptr_t) ptr;
ull * ullptr = (ull *) ptr;
iptr = (iptr / sizeof(ull)) * sizeof(ull);
- if (iptr != (int) ptr)
+ if (iptr != (uintptr_t) ptr)
return 0;
if (--ullptr < buffer || ullptr[0] > SIZE || ullptr + ullptr[0] > current)
return 0;
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2023-01-13 17:05 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-13 17:05 [newlib-cygwin] Cygwin: testsuite: Fix compilation warnings Jon Turney
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).