[#1907] addressed comments

diff --git a/src/bin/agent/main.cc b/src/bin/agent/main.cc
index 261318105f6d151b808f8cacc77325709840605a..a02f749b38224626f4f4b4a0d9732aa32f40115e 100644 (file)
--- a/src/bin/agent/main.cc
+++ b/src/bin/agent/main.cc
@@ -18,7 +18,7 @@ int main(int argc, char* argv[]) {
 
     // Launch the controller passing in command line arguments.
     // Exit program with the controller's return code.
-    try  {
+    try {
         // Instantiate/fetch the application controller singleton.
         DControllerBasePtr& controller = CtrlAgentController::instance();
 
@@ -38,6 +38,9 @@ int main(int argc, char* argv[]) {
     } catch (const std::exception& ex) {
         std::cerr << "Service failed: " << ex.what() << std::endl;
         ret = EXIT_FAILURE;
+    } catch (...) {
+        std::cerr << "Service failed: Unknown error" << std::endl;
+        ret = EXIT_FAILURE;
     }
 
     return (ret);

diff --git a/src/bin/d2/main.cc b/src/bin/d2/main.cc
index 8402fda1bef98f878df6beacd9d88752f5a839c7..4c7b317659252d59b0f4bb34d87a81666758226f 100644 (file)
--- a/src/bin/d2/main.cc
+++ b/src/bin/d2/main.cc
@@ -27,7 +27,7 @@ int main(int argc, char* argv[]) {
 
     // Launch the controller passing in command line arguments.
     // Exit program with the controller's return code.
-    try  {
+    try {
         // Instantiate/fetch the DHCP-DDNS application controller singleton.
         DControllerBasePtr& controller = D2Controller::instance();
 
@@ -47,6 +47,9 @@ int main(int argc, char* argv[]) {
     } catch (const std::exception& ex) {
         std::cerr << "Service failed: " << ex.what() << std::endl;
         ret = EXIT_FAILURE;
+    } catch (...) {
+        std::cerr << "Service failed: Unknown error" << std::endl;
+        ret = EXIT_FAILURE;
     }
 
     return (ret);

diff --git a/src/bin/dhcp4/main.cc b/src/bin/dhcp4/main.cc
index 4df7bc66620dc3ec72430a477ea1f4617ebd714b..c23bd1c6ff22cbeab3523e8acd2b268f076d395b 100644 (file)
--- a/src/bin/dhcp4/main.cc
+++ b/src/bin/dhcp4/main.cc
@@ -292,6 +292,10 @@ main(int argc, char* argv[]) {
             // Already logged so ignore
         }
         ret = EXIT_FAILURE;
+    } catch (...) {
+        cerr << DHCP4_NAME << ": Fatal error during start up: Unknown error"
+             << endl;
+        ret = EXIT_FAILURE;
     }
 
     return (ret);

diff --git a/src/bin/dhcp6/main.cc b/src/bin/dhcp6/main.cc
index 462dcea5bc5595d1d2a00deb43b39150f115947b..6ca6301a9c1a29317ff911d8079468aa18e9b59b 100644 (file)
--- a/src/bin/dhcp6/main.cc
+++ b/src/bin/dhcp6/main.cc
@@ -292,6 +292,10 @@ main(int argc, char* argv[]) {
             // Already logged so ignore
         }
         ret = EXIT_FAILURE;
+    } catch (...) {
+        cerr << DHCP6_NAME << ": Fatal error during start up: Unknown error"
+             << endl;
+        ret = EXIT_FAILURE;
     }
 
     return (ret);

diff --git a/src/bin/lfc/main.cc b/src/bin/lfc/main.cc
index 16054ac234943587bcf4df6558675bac60ee6d33..5ded1b4f606b7d055085720deef59affff323118 100644 (file)
--- a/src/bin/lfc/main.cc
+++ b/src/bin/lfc/main.cc
@@ -44,10 +44,12 @@ int main(int argc, char* argv[]) {
     } catch (const boost::exception& ex) {
         std::cerr << boost::diagnostic_information(ex) << std::endl;
         ret = EXIT_FAILURE;
-
     } catch (const std::exception& ex) {
         std::cerr << "Service failed: " << ex.what() << std::endl;
         ret = EXIT_FAILURE;
+    } catch (...) {
+        std::cerr << "Service failed: Unknown error" << std::endl;
+        ret = EXIT_FAILURE;
     }
 
     return (ret);

diff --git a/src/bin/netconf/main.cc b/src/bin/netconf/main.cc
index 02bb36f9cd2e130a1b6bc50c3063fbb034efefd8..0641779934a4a6f0663fbba43cdab2205861ad1d 100644 (file)
--- a/src/bin/netconf/main.cc
+++ b/src/bin/netconf/main.cc
@@ -38,6 +38,9 @@ int main(int argc, char* argv[]) {
     } catch (const std::exception& ex) {
         std::cerr << "Service failed: " << ex.what() << std::endl;
         ret = EXIT_FAILURE;
+    } catch (...) {
+        std::cerr << "Service failed: Unknown error" << std::endl;
+        ret = EXIT_FAILURE;
     }
 
     return (ret);

diff --git a/src/bin/perfdhcp/main.cc b/src/bin/perfdhcp/main.cc
index d7dbcfccb16c9ffe778e0566368b75e3d46eadbc..0475fff143d1522fec31ee810ce785165aa014a2 100644 (file)
--- a/src/bin/perfdhcp/main.cc
+++ b/src/bin/perfdhcp/main.cc
@@ -21,7 +21,7 @@ int
 main(int argc, char* argv[]) {
     int ret_code = 0;
     std::string diags;
-    bool parsed = false;
+    bool parser_error = true;
     try {
         CommandOptions command_options;
         diags = command_options.getDiags();
@@ -35,9 +35,7 @@ main(int argc, char* argv[]) {
         if (command_options.parse(argc, argv, true)) {
             return (ret_code);
         }
-
-        parsed = true;
-
+        parser_error = false;
         auto scenario = command_options.getScenario();
         PerfSocket socket(command_options);
         if (scenario == Scenario::BASIC) {
@@ -49,7 +47,7 @@ main(int argc, char* argv[]) {
         }
     } catch (const std::exception& e) {
         ret_code = 1;
-        if (parsed) {
+        if (!parser_error) {
             std::cerr << std::endl << "ERROR: running perfdhcp: "
                       << e.what() << std::endl;
         } else {
@@ -60,6 +58,19 @@ main(int argc, char* argv[]) {
         if (diags.find('e') != std::string::npos) {
             std::cerr << "Fatal error" << std::endl;
         }
+    } catch (...) {
+        ret_code = 1;
+        if (!parser_error) {
+            std::cerr << std::endl << "ERROR: running perfdhcp: Unknown error"
+                      << std::endl;
+        } else {
+            CommandOptions::usage();
+            std::cerr << std::endl << "ERROR: parsing command line options: Unknown error"
+                      << std::endl;
+        }
+        if (diags.find('e') != std::string::npos) {
+            std::cerr << "Fatal error" << std::endl;
+        }
     }
     return (ret_code);
 }

diff --git a/src/lib/log/compiler/message.cc b/src/lib/log/compiler/message.cc
index 27bbdb96c9734b2d2ba097c05fcd1e9badcf38a7..0359cde86e494ef9347a14376822d3689f6c3458 100644 (file)
--- a/src/lib/log/compiler/message.cc
+++ b/src/lib/log/compiler/message.cc
@@ -526,11 +526,15 @@ main(int argc, char* argv[]) {
             }
         }
 
-        cerr << text << std::endl;
+        cerr << text << "\n";
 
         return (1);
     } catch (const std::exception& ex) {
-        std::cerr << "Fatal error: " << ex.what() << std::endl;
+        cerr << "Fatal error: " << ex.what() << "\n";
+
+        return (1);
+    } catch (...) {
+        cerr << "Fatal error: Unknown error\n";
 
         return (1);
     }