Index: trunk/ppImage/src/ppImageDetrendReadout.c
===================================================================
--- trunk/ppImage/src/ppImageDetrendReadout.c	(revision 33963)
+++ trunk/ppImage/src/ppImageDetrendReadout.c	(revision 34082)
@@ -35,17 +35,25 @@
       }
     }
+
     // Check to see if we're in a chip that contains video
+    // NOTE: this is an esoteric test for a GPC1 feature : this is irrelevant & will not work for other cameras
     bool hasVideo = false;
     {
+      // XXX test for GPC1? or CAN_HAVE_VIDEO in camera config?
+      if (!input) goto done_video_check;
+      if (!input->parent) goto done_video_check;
+      if (!input->parent->parent) goto done_video_check;
+      if (!input->parent->parent->hdu) goto done_video_check;
+      if (!input->parent->parent->hdu->header) goto done_video_check;
       if (psMetadataLookupStr(NULL,input->parent->parent->hdu->header,"CELLMODE")) {
-	psWarning("VIDEO: %d %d %d\n",(int) options->hasVideo,(int) options->useVideoDark, (int) options->useVideoMask);
+	psLogMsg ("ppImage.detrend", PS_LOG_DETAIL, "VIDEO: %d %d %d\n",(int) options->hasVideo,(int) options->useVideoDark, (int) options->useVideoMask);
 	char *Vptr = strchr(psMetadataLookupStr(NULL,input->parent->parent->hdu->header,"CELLMODE"),'V');
 	if (Vptr) {
 	  hasVideo = true;
-	  psWarning("VIDEO: %d %d %d\n",(int) options->hasVideo,(int) options->useVideoDark, (int) options->useVideoMask);
+	  psLogMsg ("ppImage.detrend", PS_LOG_INFO, "VIDEO: %d %d %d\n",(int) options->hasVideo,(int) options->useVideoDark, (int) options->useVideoMask);
 	}
       }
     }
-
+    done_video_check:
     
     // Masking on the basis of pixel value needs to be done before anything else, so the values are pristine.