diff mbox

[COMMITTED] Avoid boolean coercion in tst-tls-atexit test case

Message ID 20150710180019.GA13692@spoyarek.pnq.redhat.com
State New
Headers show

Commit Message

Siddhesh Poyarekar July 10, 2015, 6 p.m. UTC
Committed as obvious.

---
 ChangeLog               | 4 ++++
 stdlib/tst-tls-atexit.c | 4 ++--
 2 files changed, 6 insertions(+), 2 deletions(-)
diff mbox

Patch

diff --git a/ChangeLog b/ChangeLog
index f4e4be6..8ee61d6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@ 
+2015-07-10  Siddhesh Poyarekar  <siddhesh@redhat.com>
+
+	* stdlib/tst-tls-atexit.c (load): Avoid boolean coercion.
+
 2015-07-10  Torvald Riegel  <triegel@redhat.com>
 
 	* nptl/sem_waitcommon.c (__new_sem_wait_slow): Update comments.
diff --git a/stdlib/tst-tls-atexit.c b/stdlib/tst-tls-atexit.c
index 68247d1..7a69722 100644
--- a/stdlib/tst-tls-atexit.c
+++ b/stdlib/tst-tls-atexit.c
@@ -36,7 +36,7 @@  load (void *u)
 {
   pthread_mutex_lock (&m);
   handle = dlopen ("$ORIGIN/tst-tls-atexit-lib.so", RTLD_LAZY);
-  if (!handle)
+  if (handle == NULL)
     {
       printf ("Unable to load DSO: %s\n", dlerror ());
       return (void *) (uintptr_t) 1;
@@ -44,7 +44,7 @@  load (void *u)
 
   void (*foo) (void) = (void (*) (void)) dlsym(handle, "do_foo");
 
-  if (!foo)
+  if (foo == NULL)
     {
       printf ("Unable to find symbol: %s\n", dlerror ());
       exit (1);