diff options
| author | Siddhesh Poyarekar <siddhesh@redhat.com> | 2015-07-10 23:23:57 +0530 |
|---|---|---|
| committer | Siddhesh Poyarekar <siddhesh@redhat.com> | 2015-07-10 23:23:57 +0530 |
| commit | 0c8f39cbf769ef35f54b57859e1e87b5291daf5e (patch) | |
| tree | 6a726ee20407fd8b55f01889b932215c477c7c8a | |
| parent | 962a1aeb28ac38019034e89ae4e87e515f3d25bd (diff) | |
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-- | ChangeLog | 3 | ||||
| -rw-r--r-- | stdlib/tst-tls-atexit.c | 3 |
2 files changed, 3 insertions, 3 deletions
@@ -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; } |