Changeset 23718 in vbox for trunk/src/VBox/Runtime/generic

Timestamp:

Oct 13, 2009 12:36:13 PM (15 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

53438

Message:

critsect-generic.cpp: Don't use AssertMsgFailed on nested entry since the logger may end up here if we assert while formatting the message for a previous assertion (happens when you get a format type wrong).

File:

• trunk/src/VBox/Runtime/generic/critsect-generic.cpp (modified) (1 diff)

trunk/src/VBox/Runtime/generic/critsect-generic.cpp

@@ -315 +315 @@
                 return VINF_SUCCESS;
             }
-            else
-            {
-                AssertMsgFailed(("Nested entry of critsect %p\n", pCritSect));
-                ASMAtomicDecS32(&pCritSect->cLockers);
-                return VERR_SEM_NESTED;
-            }
+
+            AssertBreakpoint(); /* don't do normal assertion here, the logger uses this code too. */
+            ASMAtomicDecS32(&pCritSect->cLockers);
+            return VERR_SEM_NESTED;
         }