Changeset 6386 for branches/eam_rel9_p0/psModules/src/astrom/pmChipMosaic.c

Timestamp:

Feb 8, 2006, 1:02:02 PM (20 years ago)

Author:

Paul Price

Message:

Fixing memory leaks and multiple frees

File:

• branches/eam_rel9_p0/psModules/src/astrom/pmChipMosaic.c (modified) (3 diffs)

branches/eam_rel9_p0/psModules/src/astrom/pmChipMosaic.c

@@ -288 +288 @@
     int nCells = cells->n;
 
-    // Modify the "original" pixels
+    // Fix up the HDU
     if (chip->parent->hdu) {
         psLogMsg(__func__, PS_LOG_WARN, "The original format has the entire FPA in a single extension.  "
@@ -311 +311 @@
     }
 
-    // Chop off all the component cells, and put in a new one
+    // Chop off all the component cells, and construct a new one
     pmCell *newCell = pmCellAlloc(NULL, NULL, __func__); // New cell
     cellConcepts(newCell, cells, xBinChip, yBinChip);
@@ -326 +326 @@
     // This preserves the relationship there was before, where freeing the parent frees the child.
     psRegion entire = {0.0, 0.0, 0.0, 0.0};
-    newReadout->image = psImageSubset(image, entire);
-    newReadout->weight = psImageSubset(weight, entire);
-    newReadout->mask = psImageSubset(mask, entire);
+    newReadout->image = psMemIncrRefCounter(psImageSubset(image, entire));
+    newReadout->weight = psMemIncrRefCounter(psImageSubset(weight, entire));
+    newReadout->mask = psMemIncrRefCounter(psImageSubset(mask, entire));
     // Drop references
     psFree(newReadout);