aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSiddhesh Poyarekar <siddhesh@redhat.com>2015-07-10 23:23:57 +0530
committerSiddhesh Poyarekar <siddhesh@redhat.com>2015-07-10 23:23:57 +0530
commit0c8f39cbf769ef35f54b57859e1e87b5291daf5e (patch)
tree6a726ee20407fd8b55f01889b932215c477c7c8a
parent962a1aeb28ac38019034e89ae4e87e515f3d25bd (diff)
downloadglibc-0c8f39cbf769ef35f54b57859e1e87b5291daf5e.tar
glibc-0c8f39cbf769ef35f54b57859e1e87b5291daf5e.tar.gz
glibc-0c8f39cbf769ef35f54b57859e1e87b5291daf5e.tar.bz2
glibc-0c8f39cbf769ef35f54b57859e1e87b5291daf5e.zip
Remove unnecessary mutex locks from tst-tls-atexit test case
The locks don't synchronize with anything - they were likely introduced initially to synchronize with some main thread code, but that is no longer evident.
-rw-r--r--ChangeLog3
-rw-r--r--stdlib/tst-tls-atexit.c3
2 files changed, 3 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index 8ee61d62b7..0c7fc2ca92 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
2015-07-10 Siddhesh Poyarekar <siddhesh@redhat.com>
+ * stdlib/tst-tls-atexit.c (load): Remove unnecessary mutex
+ locks.
+
* stdlib/tst-tls-atexit.c (load): Avoid boolean coercion.
2015-07-10 Torvald Riegel <triegel@redhat.com>
diff --git a/stdlib/tst-tls-atexit.c b/stdlib/tst-tls-atexit.c
index 7a69722490..5ee04a88c1 100644
--- a/stdlib/tst-tls-atexit.c
+++ b/stdlib/tst-tls-atexit.c
@@ -29,12 +29,10 @@
#include <errno.h>
void *handle;
-pthread_mutex_t m = PTHREAD_MUTEX_INITIALIZER;
void *
load (void *u)
{
- pthread_mutex_lock (&m);
handle = dlopen ("$ORIGIN/tst-tls-atexit-lib.so", RTLD_LAZY);
if (handle == NULL)
{
@@ -55,7 +53,6 @@ load (void *u)
/* This should not unload the DSO. If it does, then the thread exit will
result in a segfault. */
dlclose (handle);
- pthread_mutex_unlock (&m);
return NULL;
}