Changeset 6133

Timestamp:

Jan 21, 2006, 3:12:34 PM (20 years ago)

Author:

jhoblitt

Message:

check function parameters

Location:

trunk/ippTools/src

Files:

• chiptoolConfig.c (modified) (1 diff)
• p2insertDoneFrames.c (modified) (1 diff)
• p2insertPendingFrames.c (modified) (1 diff)
• p2pendingToDone.c (modified) (1 diff)
• p2rawToPending.c (modified) (1 diff)
• p2searchPendingFrames.c (modified) (1 diff)
• p2searchRawFrames.c (modified) (1 diff)
• p2updatePending.c (modified) (1 diff)
• p2writePendingFrames.c (modified) (2 diffs)
• pxadminConfig.c (modified) (1 diff)

trunk/ippTools/src/chiptoolConfig.c

@@ -4 +4 @@
 
 bool p2searchConfig (p2Config *config, int argc, char **argv) {
+    PS_ASSERT_PTR_NON_NULL(config, false);
 
     // Parse the configurations (re-org a la ppImage)

trunk/ippTools/src/p2insertDoneFrames.c

@@ -4 +4 @@
 // select pending frames (exposure+images) which are done/not done
 bool p2insertDoneFrames(p2Config *config, psArray *doneFrames) {
+    PS_ASSERT_PTR_NON_NULL(config, false);
+    PS_ASSERT_PTR_NON_NULL(doneFrames, false);
 
     for (int i = 0; i < doneFrames->n; i++) {

trunk/ippTools/src/p2insertPendingFrames.c

@@ -7 +7 @@
 //      this is useful since the table must be locked the whole time, to get the versions
 bool p2insertPendingFrames(p2Config *config, psArray *rawFrames) {
+    PS_ASSERT_PTR_NON_NULL(config, false);
+    PS_ASSERT_PTR_NON_NULL(rawFrames, false);
 
     psMetadata *where = psMetadataAlloc();

trunk/ippTools/src/p2pendingToDone.c

@@ -3 +3 @@
 // select pending frames (exposure+images) which are done/not done
 psArray *p2pendingToDone(p2Config *config, psArray *pendingFrames) {
+    PS_ASSERT_PTR_NON_NULL(config, NULL);
+    PS_ASSERT_PTR_NON_NULL(pendingFrames, NULL);
 
     psArray *doneFrames = psArrayAlloc(pendingFrames->n);

trunk/ippTools/src/p2rawToPending.c

@@ -3 +3 @@
 // select pending frames (exposure+images) which are done/not done
 psArray *p2rawToPending(p2Config *config, psArray *rawFrames) {
+    PS_ASSERT_PTR_NON_NULL(config, NULL);
+    PS_ASSERT_PTR_NON_NULL(rawFrames, NULL);
 
     psArray *pendingFrames = psArrayAlloc(rawFrames->n);

trunk/ippTools/src/p2searchPendingFrames.c

@@ -3 +3 @@
 // select pending frames (exposure+images) which are done/not done
 psArray *p2searchPendingFrames(p2Config *config) {
+    PS_ASSERT_PTR_NON_NULL(config, NULL);
 
     // build 'where' structure

trunk/ippTools/src/p2searchRawFrames.c

@@ -3 +3 @@
 // select raw frames (exposure+images) which match the given config options
 psArray *p2searchRawFrames(p2Config *config) {
+    PS_ASSERT_PTR_NON_NULL(config, NULL);
 
     // build 'where' structure

trunk/ippTools/src/p2updatePending.c

@@ -2 +2 @@
 
 // select pending frames (exposure+images) which are not done, select their done images, count, update
-bool p2updatePendingFrames (p2Config *config, psArray *pendingFrames) {
+bool p2updatePendingFrames(p2Config *config, psArray *pendingFrames) {
+    PS_ASSERT_PTR_NON_NULL(config, false);
+    PS_ASSERT_PTR_NON_NULL(pendingFrames, false);
 
     psMetadata *where = psMetadataAlloc ();

trunk/ippTools/src/p2writePendingFrames.c

@@ -3 +3 @@
 // select pending frames (exposure+images) which are done/not done
 bool p2writePendingFrames (p2Config *config, psArray *frames) {
+    PS_ASSERT_PTR_NON_NULL(config, false);
+    PS_ASSERT_PTR_NON_NULL(frames, false);
 
     fprintf (stdout, "# exp_id class class_id url\n");
@@ -8 +10 @@
     for (int i = 0; i < frames->n; i++) {
         p2PendingFrame *frame = frames->data[i];
+        PS_ASSERT_PTR_NON_NULL(frame, false);
+        PS_ASSERT_PTR_NON_NULL(frame->images, false);
+
         for (int j = 0; j < frame->images->n; j++) {
             p2PendingImfileRow *image = frame->images->data[j];

trunk/ippTools/src/pxadminConfig.c

@@ -2 +2 @@
 # include "p2tools.h"
 
-bool p2adminConfig (p2Config *config, int argc, char **argv) {
+bool p2adminConfig(p2Config *config, int argc, char **argv) {
+    PS_ASSERT_PTR_NON_NULL(config, false);
 
     // Parse the configurations (re-org a la ppImage)