Changeset 42806 for trunk

Timestamp:

May 8, 2025, 11:18:19 AM (14 months ago)

Author:

eugene

Message:

fix memleak in pmDark

Location:

trunk

Files:

• . (modified) (1 prop)
• Ohana (modified) (1 prop)
• Ohana/src/relphot (modified) (1 prop)
• PS-IPP-Config (modified) (1 prop)
• ippMonitor (modified) (1 prop)
• ippTools (modified) (1 prop)
• ippTools/src (modified) (1 prop)
• ippconfig (modified) (1 prop)
• ppImage (modified) (1 prop)
• psLib (modified) (1 prop)
• psModules (modified) (1 prop)
• psModules/src/detrend/pmDark.c (modified) (5 diffs)
• psconfig (modified) (1 prop)

trunk/psModules/src/detrend/pmDark.c

@@ -252 +252 @@
     }
 
+# if (PS_TRACE_ON)
     if (psTraceGetLevel("psModules.detrend") > 9) {
         for (int i = 0; i < inputs->n; i++) {
             psVector *val = values->data[i];
-            (void) val; // avoid unused variable message when tracing is compiled out
             for (int j = 0; j < ordinates->n; j++) {
                 psTrace("psModules.detrend", 9, "Image %d, ordinate %d: %f\n", i, j, val->data.F32[j]);
@@ -261 +261 @@
         }
     }
+# endif
 
     int numTerms = 1;                   // Number of terms in polynomial
@@ -292 +293 @@
         pmReadout *readout = output->readouts->data[i]; // Readout to update
         if (!readout) {
-            readout = output->readouts->data[i] = pmReadoutAlloc(output);
-        }
-
+            // pmReadoutAlloc places a reference to the readout on cell->readouts
+            // and returns a second mem reference to the readout
+            readout = pmReadoutAlloc(output);
+            psFree (readout); // free the extra mem reference returned by pmReadoutAlloc
+        }
         pmReadoutStackDefineOutput(readout, col0, row0, numCols, numRows, false, false, 0);
         psTrace("psModules.imcombine", 7, "Output minimum: %d,%d\n", col0, row0);
@@ -469 +472 @@
               for (int m = 0; m < poly->deviations->n; m++) {
                 logL->data.F64[k] += pow(poly->deviations->data.F32[m] / polySig->stdevFit,2);
-/*              if ((xOut == 20) && (yOut == 256)) { */
-/*                psTrace("psModules.detrend",3,"pmDarkCombine DEV: %d %d: input %d models: Norders: %d logL: %g value: %g\n", */
-/*                        xOut,yOut,m,k,logL->data.F64[k],poly->deviations->data.F32[m]); */
-/*              } */
               }
               if (k > 0) {
@@ -487 +486 @@
               k_best = 1;
             }
-/*          k_best = 1; */
+
             // Select the polynomial that seems best.
             psPolynomialMD *poly = poly_set->data[k_best];
               
-            //            for (int k = 0; k < poly->coeff->n; k++) {
             for (int k = 0; k < max_orders->n + 1; k++) { // There is one more coefficient than is stored here.
                 pmReadout *ro = output->readouts->data[k]; // Readout of interest
                 if (k < poly->coeff->n) {
                   ro->image->data.F32[yOut][xOut] = poly->coeff->data.F64[k];
-                }
-                else {
+                } else {
                   ro->image->data.F32[yOut][xOut] = 0.0;
                 }