Changeset 26834

Timestamp:

Feb 9, 2010, 6:24:04 PM (16 years ago)

Author:

Paul Price

Message:

Catch SEGV.

File:

• branches/eam_branches/20091201/psModules/src/imcombine/pmPSFEnvelope.c (modified) (3 diffs)

branches/eam_branches/20091201/psModules/src/imcombine/pmPSFEnvelope.c

@@ -66 +66 @@
                      const char *modelName,// Name of PSF model to use
                      int xOrder, int yOrder, // Order for PSF variation fit
-                     psImageMaskType maskVal
+                     psImageMaskType maskVal
                      )
 {
@@ -155 +155 @@
             pmModelClassSetLimits(PM_MODEL_LIMITS_IGNORE);
             pmModel *model = pmModelFromPSFforXY(psf, numCols / 2.0, numRows / 2.0, PEAK_FLUX); // Test model
-            model->modelSetLimits(PM_MODEL_LIMITS_STRICT);
-            for (int j = 0; j < model->params->n && goodPSF; j++) {
-                if (!model->modelLimits(PS_MINIMIZE_PARAM_MIN, j, model->params->data.F32, NULL) ||
-                    !model->modelLimits(PS_MINIMIZE_PARAM_MAX, j, model->params->data.F32, NULL)) {
-                    goodPSF = false;
+            if (!model) {
+                goodPSF = false;
+            } else {
+                model->modelSetLimits(PM_MODEL_LIMITS_STRICT);
+                for (int j = 0; j < model->params->n && goodPSF; j++) {
+                    if (!model->modelLimits(PS_MINIMIZE_PARAM_MIN, j, model->params->data.F32, NULL) ||
+                        !model->modelLimits(PS_MINIMIZE_PARAM_MAX, j, model->params->data.F32, NULL)) {
+                        goodPSF = false;
+                    }
                 }
-            }
-            psFree(model);
+                psFree(model);
+            }
             if (!goodPSF) {
                 psWarning("PSF %d is bad --- not including in envelope calculation.", i);
@@ -361 +365 @@
 
         // measure the source moments: tophat windowing, no pixel S/N cutoff
-        // XXX probably should be passing the maskVal to this function so we can pass it along here...
+        // XXX probably should be passing the maskVal to this function so we can pass it along here...
         if (!pmSourceMoments(source, maxRadius, 0.0, 1.0, maskVal)) {
             // Can't do anything about it; limp along as best we can