Merge pull request #3433 from matt335672/coverity_scan

Address Coverity mutex issues
2025-02-14 17:09:53 +00:00
parent fa9cc88389 67fbccc539
commit 6889a0731d
1 changed files with 10 additions and 3 deletions
@@ -33,6 +33,7 @@
 #include <semaphore.h>
 #endif
 #include "arch.h"
+#include "log.h"
 #include "thread_calls.h"
 #include "os_calls.h"

@@ -136,11 +137,17 @@ tc_mutex_lock(tbus mutex)
 {
 #if defined(_WIN32)
    WaitForSingleObject((HANDLE)mutex, INFINITE);
-    return 0;
 #else
+    if (mutex != 0)
+    {
        pthread_mutex_lock((pthread_mutex_t *)mutex);
-    return 0;
+    }
+    else
+    {
+        LOG(LOG_LEVEL_ERROR, "Attempt made to lock NULL mutex");
+    }
 #endif
+    return 0;
 }

 /*****************************************************************************/