Changeset 21294 for branches/eam_branch_20090203/psLib/src/imageops/psImageStructManip.c

Timestamp:

Feb 4, 2009, 1:36:56 PM (17 years ago)

Author:

eugene

Message:

adding mutex locks to psImage structure manipulation

File:

• branches/eam_branch_20090203/psLib/src/imageops/psImageStructManip.c (modified) (7 diffs)

branches/eam_branch_20090203/psLib/src/imageops/psImageStructManip.c

@@ -8 +8 @@
  *  @author Robert DeSonia, MHPCC
  *
- *  @version $Revision: 1.20 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2007-11-08 01:09:48 $
+ *  @version $Revision: 1.20.42.1 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2009-02-04 23:36:56 $
  *
  *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -21 +21 @@
 #include <string.h>
 #include <assert.h>
+#include <errno.h>
 
 #include "psMemory.h"
 #include "psError.h"
 #include "psAbort.h"
+#include "psTrace.h"
 
 #include "psImageStructManip.h"
@@ -100 +102 @@
     psPtr rawData = psMemIncrRefCounter(image->p_rawDataBuffer);
 
+    // XXX worry about this later...
     if (out != NULL) {
         // if a child, need to orphan (disassociate from parent) first
@@ -108 +111 @@
 
             // drop my entry on my parent's array of children
+            psMutexLock (out);
             psArrayRemoveDataNoFree (out->parent->children, out);
+            psMutexUnlock (out);
 
             // drop my reference to my old parent
@@ -119 +124 @@
         out = p_psAlloc(file, lineno, func, sizeof(psImage));
         out->data.V = NULL;
+        psMutexInit (out);
     }
 
@@ -129 +135 @@
     P_PSIMAGE_SET_ROW0(out, row0);
 
+    // As long as I have a valid image reference, no one else can free it to zero (I have at
+    // least the last reference)
     out->parent = psMemIncrRefCounter(image); // track references to parents
     out->children = NULL;
@@ -144 +152 @@
     }
 
-    // add output image as a child of the input image.
+    // Add output image as a child of the input image.  Lock image before performing this
+    // operation (psArrayAdd is NOT thread-safe)
+    psMutexLock (image);
     image->children = p_psArrayAdd(file, lineno, func, image->children, 16, out);
+    psMutexUnlock (image);
+
     psFree (out); // the image->children array is an array of views only
-
     return (out);
 }