Changeset 25660 in vbox for trunk/src/VBox/Runtime/common

Timestamp:

Jan 5, 2010 4:52:08 PM (15 years ago)

Author:

vboxsync

Message:

thread.cpp: Fixed case of accessing RTTHRADINT after it was freed.

File:

• trunk/src/VBox/Runtime/common/misc/thread.cpp (modified) (5 diffs)

trunk/src/VBox/Runtime/common/misc/thread.cpp

@@ -524 +524 @@
 }
 
-
 /**
  * Release a per thread data structure.
@@ -542 +541 @@
     }
     else
+    {
         cRefs = 0;
+        AssertFailed();
+    }
     return cRefs;
 }
@@ -571 +573 @@
 
     /*
-     * Invalidate the thread structure and free it.
+     * Invalidate the thread structure.
      */
 #ifdef IN_RING3
@@ -585 +587 @@
     pThread->EventTerminated = NIL_RTSEMEVENTMULTI;
 
-    RTMemFree(pThread);
-
 #ifdef IN_RING3
     rtLockValidatorSerializeDestructLeave();
@@ -592 +592 @@
 
     /*
-     * Destroy semaphore resources.
+     * Destroy semaphore resources and free the bugger.
      */
     RTSemEventMultiDestroy(hEvt1);
     if (hEvt2 != NIL_RTSEMEVENTMULTI)
         RTSemEventMultiDestroy(hEvt2);
+
+    RTMemFree(pThread);
 }