[libvirt] [PATCH] Fix leak of mutex attributes in POSIX threads impl

Daniel P. Berrange berrange at redhat.com
Mon Jan 24 15:35:38 UTC 2011


* src/util/threads-pthread.c: Fix mutex leak
---
 src/util/threads-pthread.c |    8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/util/threads-pthread.c b/src/util/threads-pthread.c
index ea71589..f812045 100644
--- a/src/util/threads-pthread.c
+++ b/src/util/threads-pthread.c
@@ -47,7 +47,9 @@ int virMutexInit(virMutexPtr m)
     pthread_mutexattr_t attr;
     pthread_mutexattr_init(&attr);
     pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_NORMAL);
-    if ((ret = pthread_mutex_init(&m->lock, &attr)) != 0) {
+    ret = pthread_mutex_init(&m->lock, &attr);
+    pthread_mutexattr_destroy(&attr);
+    if (ret != 0) {
         errno = ret;
         return -1;
     }
@@ -60,7 +62,9 @@ int virMutexInitRecursive(virMutexPtr m)
     pthread_mutexattr_t attr;
     pthread_mutexattr_init(&attr);
     pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_RECURSIVE);
-    if ((ret = pthread_mutex_init(&m->lock, &attr)) != 0) {
+    ret = pthread_mutex_init(&m->lock, &attr);
+    pthread_mutexattr_destroy(&attr);
+    if (ret != 0) {
         errno = ret;
         return -1;
     }
-- 
1.7.3.4




More information about the libvir-list mailing list