about summary refs log tree commit diff
path: root/nptl/tst-mutex1.c
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2014-12-02 22:33:57 +0000
committerJoseph Myers <joseph@codesourcery.com>2014-12-02 22:33:57 +0000
commit4e0b901601038c863b97e68b18ae50a82e10a157 (patch)
tree09fb81d7961a889873ca4e76b07c20eddffd13ae /nptl/tst-mutex1.c
parentb1aff6a41120c1d9cedc1ecedeafc79f161ce982 (diff)
downloadglibc-4e0b901601038c863b97e68b18ae50a82e10a157.tar.gz
glibc-4e0b901601038c863b97e68b18ae50a82e10a157.tar.xz
glibc-4e0b901601038c863b97e68b18ae50a82e10a157.zip
Fix -Waddress warnings in nptl/tst-mutex1.c.
This patch fixes -Waddress warnings in nptl/tst-mutex1.c from
comparing the address of an object with NULL (ATTR may either be NULL,
or the address of an object when included from other tests, and the
warning arises in the latter case).  A macro ATTR_NULL is defined
alongside ATTR and used for the tests.

Tested for x86_64.

	* nptl/tst-mutex1.c: Include <stdbool.h>.
	[!ATTR] (ATTR_NULL): New macro.
	(do_test): Test !ATTR_NULL instead of ATTR != NULL.
	* nptl/tst-mutexpi1.c (ATTR_NULL): New macro.
	* nptl/tst-mutexpp1.c (ATTR_NULL): New macro.
Diffstat (limited to 'nptl/tst-mutex1.c')
-rw-r--r--nptl/tst-mutex1.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/nptl/tst-mutex1.c b/nptl/tst-mutex1.c
index 2a181e587e..27ce54f33a 100644
--- a/nptl/tst-mutex1.c
+++ b/nptl/tst-mutex1.c
@@ -19,10 +19,12 @@
 #include <pthread.h>
 #include <stdio.h>
 #include <errno.h>
+#include <stdbool.h>
 
 
 #ifndef ATTR
 # define ATTR NULL
+# define ATTR_NULL true
 #endif
 
 
@@ -32,7 +34,7 @@ do_test (void)
   pthread_mutex_t m;
 
   int e = pthread_mutex_init (&m, ATTR);
-  if (ATTR != NULL && e == ENOTSUP)
+  if (!ATTR_NULL && e == ENOTSUP)
     {
       puts ("cannot support selected type of mutexes");
       return 0;
@@ -43,7 +45,7 @@ do_test (void)
       return 1;
     }
 
-  if (ATTR != NULL && pthread_mutexattr_destroy (ATTR) != 0)
+  if (!ATTR_NULL && pthread_mutexattr_destroy (ATTR) != 0)
     {
       puts ("mutexattr_destroy failed");
       return 1;