Changeset 11857

Timestamp:

Feb 16, 2007, 12:24:20 PM (19 years ago)

Author:

jhoblitt

Message:

properly handle pzgetexpConfig() errors

Location:

trunk/ippTools/src

Files:

• pzgetexp.c (modified) (3 diffs)
• pzgetexpConfig.c (modified) (4 diffs)

trunk/ippTools/src/pzgetexp.c

@@ -39 +39 @@
 
     pxConfig *config = pzgetexpConfig(NULL, argc, argv);
+    if (!config) {
+        psError(PXTOOLS_ERR_CONFIG, false, "failed to configure");
+        goto FAIL;
+    }
 
     if (!go(config)) {
@@ -52 +56 @@
 FAIL:
     psErrorStackPrint(stderr, "\n");
+    int exit_status = pxerrorGetExitStatus();
     
     psFree(config);
@@ -57 +62 @@
     psLibFinalize();
 
-    exit(EXIT_FAILURE);
+    exit(exit_status);
 }
 

trunk/ippTools/src/pzgetexpConfig.c

@@ -29 +29 @@
 #include "pxtools.h"
 
-pxConfig *pzgetexpConfig(pxConfig *config, int argc, char **argv) {
+pxConfig *pzgetexpConfig(pxConfig *config, int argc, char **argv)
+{
     if (!config) {
         config = pxConfigAlloc();
@@ -39 +40 @@
     if (! config->modules) {
         psError(PS_ERR_UNKNOWN, false, "Can't find site configuration!\n");
-        exit(EXIT_FAILURE);
+        psFree(config);
+        return NULL;
     }
 
@@ -63 +65 @@
         psArgumentHelp(args);
         psFree(config);
-        exit(EXIT_FAILURE);
+        return NULL;
     }
 
@@ -75 +77 @@
         psError(PS_ERR_UNKNOWN, false, "Can't connect to db\n");
         psFree(config);
-        exit(EXIT_FAILURE);
+        return NULL;
     }