Theoretical memory leak where if attr_init failed, then check would have still been...

diff --git a/libs/esl/src/esl_threadmutex.c b/libs/esl/src/esl_threadmutex.c
index 34fa6bb96a72d689327eb9cd30a6de7d5557a0d5..e3706e12740815130dabdfe9ea7669a8064b4625 100644 (file)
--- a/libs/esl/src/esl_threadmutex.c
+++ b/libs/esl/src/esl_threadmutex.c
@@ -148,8 +148,10 @@ ESL_DECLARE(esl_status_t) esl_mutex_create(esl_mutex_t **mutex)
 #ifdef WIN32
        InitializeCriticalSection(&check->mutex);
 #else
-       if (pthread_mutexattr_init(&attr))
+       if (pthread_mutexattr_init(&attr)) {
+               free(check);
                goto done;
+       }
 
        if (pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_RECURSIVE))
                goto fail;