diff options
| author | Frédéric Bérat <fberat@redhat.com> | 2023-06-01 16:27:47 +0200 |
|---|---|---|
| committer | Siddhesh Poyarekar <siddhesh@sourceware.org> | 2023-06-01 13:01:32 -0400 |
| commit | 29e25f6f136182fb3756d51e03dea7c4d1919dd9 (patch) | |
| tree | 4d3950068c9fec569ae21055641b986632371fbb /sysdeps | |
| parent | a952fcda58cd7aa191140fc9e7d453df212b9117 (diff) | |
| download | glibc-29e25f6f136182fb3756d51e03dea7c4d1919dd9.zip glibc-29e25f6f136182fb3756d51e03dea7c4d1919dd9.tar.gz glibc-29e25f6f136182fb3756d51e03dea7c4d1919dd9.tar.bz2 | |
tests: fix warn unused results
With fortification enabled, few function calls return result need to be
checked, has they get the __wur macro enabled.
Reviewed-by: Siddhesh Poyarekar <siddhesh@sourceware.org>
Diffstat (limited to 'sysdeps')
| -rw-r--r-- | sysdeps/pthread/tst-cancel16.c | 6 | ||||
| -rw-r--r-- | sysdeps/pthread/tst-cancel4.c | 6 |
2 files changed, 9 insertions, 3 deletions
diff --git a/sysdeps/pthread/tst-cancel16.c b/sysdeps/pthread/tst-cancel16.c index 511b9e1..d47c7c6 100644 --- a/sysdeps/pthread/tst-cancel16.c +++ b/sysdeps/pthread/tst-cancel16.c @@ -50,7 +50,11 @@ tf (void *arg) pthread_cleanup_push (cl, NULL); /* This call should never return. */ - (void) lockf (fd, F_LOCK, 0); + if (lockf (fd, F_LOCK, 0)) + { + puts ("child thread: lockf failed"); + exit (1); + } pthread_cleanup_pop (0); diff --git a/sysdeps/pthread/tst-cancel4.c b/sysdeps/pthread/tst-cancel4.c index 4f5c893..4c9e867 100644 --- a/sysdeps/pthread/tst-cancel4.c +++ b/sysdeps/pthread/tst-cancel4.c @@ -1009,7 +1009,8 @@ tf_pread (void *arg) pthread_cleanup_push (cl, NULL); char mem[10]; - pread (tempfd, mem, sizeof (mem), 0); + if (pread (tempfd, mem, sizeof (mem), 0) < 0) + FAIL_EXIT1 ("pread failed: %m"); pthread_cleanup_pop (0); @@ -1038,7 +1039,8 @@ tf_pwrite (void *arg) pthread_cleanup_push (cl, NULL); char mem[10]; - pwrite (tempfd, mem, sizeof (mem), 0); + if (pwrite (tempfd, mem, sizeof (mem), 0) <0) + FAIL_EXIT1 ("pwrite failed: %m"); pthread_cleanup_pop (0); |