Changeset 48035 in vbox

Timestamp:

Aug 23, 2013 5:29:18 PM (11 years ago)

Author:

vboxsync

Message:

Runtime/threadctxhooks: Paranoia.

File:

• trunk/src/VBox/Runtime/r0drv/solaris/threadctxhooks-r0drv-solaris.c (modified) (4 diffs)

trunk/src/VBox/Runtime/r0drv/solaris/threadctxhooks-r0drv-solaris.c

@@ -37 +37 @@
 #include <iprt/err.h>
 #include <iprt/asm.h>
+#include <iprt/log.h>
 #include "internal/thread.h"
 
@@ -57 +58 @@
     void                       *pvUser;
     /** Whether this handle has any hooks registered or not. */
-    bool                        fRegistered;
+    bool volatile               fRegistered;
     /** Number of references to this object. */
     uint32_t volatile           cRefs;
@@ -213 +214 @@
     Assert(RTThreadPreemptIsEnabled(NIL_RTTHREAD));
 
-    pThis->fRegistered = false;
+    ASMAtomicWriteBool(&pThis->fRegistered, false);
     uint32_t cRefs = ASMAtomicDecU32(&pThis->cRefs);
 
@@ -251 +252 @@
                                                           rtThreadCtxHooksSolFree);
         }
-        AssertMsg(rc, ("removectx failed. rc=%d\n", rc));
+        AssertMsg(rc, ("removectx() failed. rc=%d\n", rc));
         NOREF(rc);