Changeset 10894 for branches/jch-memory/psLib/src/sys/psMemory.c

Timestamp:

Jan 3, 2007, 12:04:42 PM (19 years ago)

Author:

jhoblitt

Message:

change psMemCheckCorruption() to hold the memBlockList mutex the entire time it is invoked

File:

• branches/jch-memory/psLib/src/sys/psMemory.c (modified) (2 diffs)

branches/jch-memory/psLib/src/sys/psMemory.c

@@ -8 +8 @@
 *  @author Robert Lupton, Princeton University
 *
-*  @version $Revision: 1.88.2.6 $ $Name: not supported by cvs2svn $
-*  @date $Date: 2007-01-03 21:55:11 $
+*  @version $Revision: 1.88.2.7 $ $Name: not supported by cvs2svn $
+*  @date $Date: 2007-01-03 22:04:42 $
 *
 *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -305 +305 @@
     psBool failure = false;
 
-    // get exclusive access to the memBlock list to avoid it changing on us while we use it.
-    //    MUTEX_LOCK(&memBlockListMutex);
+    // get exclusive access to the memBlock list to avoid it changing on us
+    // while we use it.
+    MUTEX_LOCK(&memBlockListMutex);
 
     for (psMemBlock* iter = lastMemBlockAllocated; iter != NULL; iter = iter->nextBlock) {
-        MUTEX_UNLOCK(&memBlockListMutex);
-        failure = checkMemBlock(iter, __func__);
-        MUTEX_LOCK(&memBlockListMutex);
-        if ( failure ) {
+        if (checkMemBlock(iter, __func__)) {
             nbad++;