Security-57740.20.22.tar.gz

[apple/security.git] / OSX / libsecurity_utilities / lib / threading.cpp

diff --git a/OSX/libsecurity_utilities/lib/threading.cpp b/OSX/libsecurity_utilities/lib/threading.cpp
index bfe6427ca92ddf131ca8de151cbd1dce8bb2405b..4146c3f4dfdee2b0498f62c3dcd5c87289c15e1e 100644 (file)
--- a/OSX/libsecurity_utilities/lib/threading.cpp
+++ b/OSX/libsecurity_utilities/lib/threading.cpp
@@ -28,6 +28,7 @@
 #include <security_utilities/threading.h>
 #include <security_utilities/globalizer.h>
 #include <security_utilities/memutils.h>
+#include <utilities/debugging.h>
 
 #include <unistd.h>     // WWDC 2007 thread-crash workaround
 #include <syslog.h>     // WWDC 2007 thread-crash workaround
@@ -91,6 +92,9 @@ Mutex::Mutex(Type type)
 Mutex::~Mutex()
 {
     int result = pthread_mutex_destroy(&me);
+    if(result) {
+        secerror("Probable bug: error destroying Mutex: %d", result);
+    }
        check(result);
 }
 
@@ -156,7 +160,6 @@ void CountingMutex::enter()
 {
     lock();
     mCount++;
-    secdebug("cmutex", "%p up to %d", this, mCount);
     unlock();
 }
 
@@ -165,7 +168,6 @@ bool CountingMutex::tryEnter()
     if (!tryLock())
         return false;
     mCount++;
-    secdebug("cmutex", "%p up to %d (was try)", this, mCount);
     unlock();
     return true;
 }
@@ -175,14 +177,12 @@ void CountingMutex::exit()
     lock();
     assert(mCount > 0);
     mCount--;
-    secdebug("cmutex", "%p down to %d", this, mCount);
     unlock();
 }
 
 void CountingMutex::finishEnter()
 {
     mCount++;
-    secdebug("cmutex", "%p finish up to %d", this, mCount);
     unlock();
 }
 
@@ -190,7 +190,6 @@ void CountingMutex::finishExit()
 {
     assert(mCount > 0);
     mCount--; 
-    secdebug("cmutex", "%p finish down to %d", this, mCount);
     unlock();
 }
 
@@ -275,7 +274,7 @@ void Thread::run()
         syslog(LOG_ERR, "too many failed pthread_create() attempts");
     }
     else
-        secdebug("thread", "%p created", self.mIdent);
+        secinfo("thread", "%p created", self.mIdent);
 }
 
 void *Thread::runner(void *arg)
@@ -284,9 +283,9 @@ void *Thread::runner(void *arg)
         // otherwise it will crash if something underneath throws.
     {
         Thread *me = static_cast<Thread *>(arg);
-        secdebug("thread", "%p starting", me->self.mIdent);
+        secinfo("thread", "%p starting", me->self.mIdent);
         me->action();
-        secdebug("thread", "%p terminating", me->self.mIdent);
+        secinfo("thread", "%p terminating", me->self.mIdent);
         delete me;
         return NULL;
     }