Changeset 11860 for trunk/ippTools/src/pztoolConfig.c

Timestamp:

Feb 16, 2007, 12:29:15 PM (19 years ago)

Author:

jhoblitt

Message:

properly handle pztoolConfig() errors

File:

• trunk/ippTools/src/pztoolConfig.c (modified) (8 diffs)

trunk/ippTools/src/pztoolConfig.c

@@ -27 +27 @@
 #include "pztool.h"
 
-// this function can not fail -- exits on error
-pxConfig *pztoolConfig(pxConfig *config, int argc, char **argv) {
+pxConfig *pztoolConfig(pxConfig *config, int argc, char **argv)
+{
     if (!config) {
         config = pxConfigAlloc();
@@ -39 +39 @@
     if (! config->modules) {
         psError(PS_ERR_UNKNOWN, false, "Can't find site configuration!\n");
-        exit(EXIT_FAILURE);
+        psFree(config);
+        return NULL;
     }
 
@@ -103 +104 @@
         if (config->mode) { \
             psError(PS_ERR_UNKNOWN, true, "only one mode selection is allowed"); \
-            goto FAIL; \
+            psFree(config); \
+            return NULL; \
         } \
         config->mode = modeval; \
@@ -148 +150 @@
 
         psFree(argSets);
-        goto FAIL;
+        psFree(config);
+        return NULL;
     }
 
@@ -161 +164 @@
         if (!psMetadataAddStr(config->where, PS_LIST_TAIL, #name, 0, "==", str)) {\
             psError(PS_ERR_UNKNOWN, false, "failed to add item " #name); \
-            goto FAIL; \
+            psFree(config); \
+            return NULL; \
         } \
     } \
@@ -178 +182 @@
             if (!psMetadataAddStr(config->where, PS_LIST_TAIL, "camera", 0, "==", str)) {
                 psError(PS_ERR_UNKNOWN, false, "failed to add item camera");
-                goto FAIL;
+                psFree(config);
+                return NULL;
             }
         }
@@ -197 +202 @@
     if (!config->dbh) {
         psError(PS_ERR_UNKNOWN, false, "Can't configure database");
-        goto FAIL;
+        psFree(config);
+        return NULL;
     }
 
@@ -205 +211 @@
 
     return config;
-
-FAIL:
-    psFree(config);
-    pmConfigDone();
-    psLibFinalize();
-    exit(EXIT_FAILURE);
 }