Fix org#2656 About incorrect error message on TLS CA Certificate

diff --git a/bacula/src/console/bbconsjson.c b/bacula/src/console/bbconsjson.c
index ba3df87ec19e7a77799616a9a5c54936f5e2e52e..0aadaa16a6efe0d6a4d0c2633ebc4308e8b36528 100644 (file)
--- a/bacula/src/console/bbconsjson.c
+++ b/bacula/src/console/bbconsjson.c
@@ -458,7 +458,7 @@ static int check_resources()
       tls_needed = director->tls_enable || director->tls_authenticate;
 
       if ((!director->tls_ca_certfile && !director->tls_ca_certdir) && tls_needed) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Director \"%s\" in %s."
                              " At least one CA certificate store is required.\n"),
                              director->hdr.name, configfile);
@@ -487,7 +487,7 @@ static int check_resources()
       }
       tls_needed = cons->tls_enable || cons->tls_authenticate;
       if ((!cons->tls_ca_certfile && !cons->tls_ca_certdir) && tls_needed) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Console \"%s\" in %s.\n"),
                              cons->hdr.name, configfile);
          OK = false;

diff --git a/bacula/src/console/console.c b/bacula/src/console/console.c
index 7b472b2c826e08caf2e6e17ad3e0bad24b48b855..b053d7df3de0370d103e4272bdb6d66feeec2c7a 100644 (file)
--- a/bacula/src/console/console.c
+++ b/bacula/src/console/console.c
@@ -1495,7 +1495,7 @@ static int check_resources()
       tls_needed = director->tls_enable || director->tls_authenticate;
 
       if ((!director->tls_ca_certfile && !director->tls_ca_certdir) && tls_needed) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Director \"%s\" in %s."
                              " At least one CA certificate store is required.\n"),
                              director->hdr.name, configfile);
@@ -1529,7 +1529,7 @@ static int check_resources()
       }
       tls_needed = cons->tls_enable || cons->tls_authenticate;
       if ((!cons->tls_ca_certfile && !cons->tls_ca_certdir) && tls_needed) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Console \"%s\" in %s.\n"),
                              cons->hdr.name, configfile);
          OK = false;

diff --git a/bacula/src/dird/bdirjson.c b/bacula/src/dird/bdirjson.c
index 9ae80542c1b78ff67a4391568902e1c4057d888e..c7dce43f87156e6d1a5003e15fc3362981beb65f 100644 (file)
--- a/bacula/src/dird/bdirjson.c
+++ b/bacula/src/dird/bdirjson.c
@@ -1226,7 +1226,7 @@ static bool check_resources(bool apply_jobdefs)
 
       if ((!director->tls_ca_certfile && !director->tls_ca_certdir) &&
            need_tls && director->tls_verify_peer) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\" or \"TLS CA"
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\" or \"TLS CA"
               " Certificate Dir\" are defined for Director \"%s\" in %s."
               " At least one CA certificate store is required"
               " when using \"TLS Verify Peer\".\n"),
@@ -1267,7 +1267,7 @@ static bool check_resources(bool apply_jobdefs)
 
       if ((!cons->tls_ca_certfile && !cons->tls_ca_certdir)
             && need_tls && cons->tls_verify_peer) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\" or \"TLS CA"
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\" or \"TLS CA"
             " Certificate Dir\" are defined for Console \"%s\" in %s."
             " At least one CA certificate store is required"
             " when using \"TLS Verify Peer\".\n"),
@@ -1313,7 +1313,7 @@ static bool check_resources(bool apply_jobdefs)
       }
       need_tls = client->tls_enable || client->tls_authenticate;
       if ((!client->tls_ca_certfile && !client->tls_ca_certdir) && need_tls) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
             " or \"TLS CA Certificate Dir\" are defined for File daemon \"%s\" in %s.\n"),
             client->name(), configfile);
          OK = false;

diff --git a/bacula/src/dird/dird.c b/bacula/src/dird/dird.c
index 338a4c6a459904a96e1819226f55ec432ce018cc..0ec020e745270c938e6a5e74efbd9203c325912b 100644 (file)
--- a/bacula/src/dird/dird.c
+++ b/bacula/src/dird/dird.c
@@ -966,7 +966,7 @@ static bool check_resources()
 
       if ((!newDirector->tls_ca_certfile && !newDirector->tls_ca_certdir) &&
            need_tls && newDirector->tls_verify_peer) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\" or \"TLS CA"
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\" or \"TLS CA"
               " Certificate Dir\" are defined for Director \"%s\" in %s."
               " At least one CA certificate store is required"
               " when using \"TLS Verify Peer\".\n"),
@@ -1199,7 +1199,7 @@ static bool check_resources()
 
       if ((!cons->tls_ca_certfile && !cons->tls_ca_certdir)
             && need_tls && cons->tls_verify_peer) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\" or \"TLS CA"
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\" or \"TLS CA"
             " Certificate Dir\" are defined for Console \"%s\" in %s."
             " At least one CA certificate store is required"
             " when using \"TLS Verify Peer\".\n"),
@@ -1256,7 +1256,7 @@ static bool check_resources()
       }
       need_tls = client->tls_enable || client->tls_authenticate;
       if ((!client->tls_ca_certfile && !client->tls_ca_certdir) && need_tls) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
             " or \"TLS CA Certificate Dir\" are defined for File daemon \"%s\" in %s.\n"),
             client->name(), configfile);
          OK = false;
@@ -1589,7 +1589,7 @@ static bool check_catalog(cat_op mode)
          need_tls = store->tls_enable || store->tls_authenticate;
 
          if ((!store->tls_ca_certfile && !store->tls_ca_certdir) && need_tls) {
-            Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+            Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                  " or \"TLS CA Certificate Dir\" are defined for Storage \"%s\" in %s.\n"),
                  store->name(), configfile);
             OK = false;

diff --git a/bacula/src/filed/bfdjson.c b/bacula/src/filed/bfdjson.c
index 78ad7225eb97d40d8baca7dbb50f1610184f0aed..76bfd78ae1e7ea24e9c48bfd66efb1d67c5fb6a2 100644 (file)
--- a/bacula/src/filed/bfdjson.c
+++ b/bacula/src/filed/bfdjson.c
@@ -624,7 +624,7 @@ static bool check_resources()
       need_tls = me->tls_enable || me->tls_authenticate;
 
       if ((!me->tls_ca_certfile && !me->tls_ca_certdir) && need_tls) {
-         Emsg1(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg1(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
             " or \"TLS CA Certificate Dir\" are defined for File daemon in %s.\n"),
                             configfile);
         OK = false;
@@ -682,7 +682,7 @@ static bool check_resources()
       }
 
       if ((!director->dirinfo.tls_ca_certfile && !director->dirinfo.tls_ca_certdir) && need_tls && director->dirinfo.tls_verify_peer) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Director \"%s\" in %s."
                              " At least one CA certificate store is required"
                              " when using \"TLS Verify Peer\".\n"),
@@ -719,7 +719,7 @@ static bool check_resources()
       }
 
       if ((!cons->dirinfo.tls_ca_certfile && !cons->dirinfo.tls_ca_certdir) && need_tls && cons->dirinfo.tls_verify_peer) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Console \"%s\" in %s."
                              " At least one CA certificate store is required"
                              " when using \"TLS Verify Peer\".\n"),

diff --git a/bacula/src/filed/filed.c b/bacula/src/filed/filed.c
index 72fdcd4a5506c01feaf7eb4a1bf0089f21d8721b..6397c07b886ea102efbc28dd10371920f51e56ef 100644 (file)
--- a/bacula/src/filed/filed.c
+++ b/bacula/src/filed/filed.c
@@ -441,7 +441,7 @@ static bool check_resources()
       need_tls = me->tls_enable || me->tls_authenticate;
 
       if ((!me->tls_ca_certfile && !me->tls_ca_certdir) && need_tls) {
-         Emsg1(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg1(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
             " or \"TLS CA Certificate Dir\" are defined for File daemon in %s.\n"),
                             configfile);
         OK = false;
@@ -666,7 +666,7 @@ static bool check_resources()
       }
 
       if ((!director->dirinfo.tls_ca_certfile && !director->dirinfo.tls_ca_certdir) && need_tls && director->dirinfo.tls_verify_peer) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Director \"%s\" in %s."
                              " At least one CA certificate store is required"
                              " when using \"TLS Verify Peer\".\n"),
@@ -731,7 +731,7 @@ static bool check_resources()
       }
 
       if ((!console->dirinfo.tls_ca_certfile && !console->dirinfo.tls_ca_certdir) && need_tls && console->dirinfo.tls_verify_peer) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Console \"%s\" in %s."
                              " At least one CA certificate store is required"
                              " when using \"TLS Verify Peer\".\n"),

diff --git a/bacula/src/qt-console/main.cpp b/bacula/src/qt-console/main.cpp
index a88435df2801d22fc957ef979b601e3fdeeb20eb..cd52c356a6a28c039d0770e6f11c387e23c25b42 100644 (file)
--- a/bacula/src/qt-console/main.cpp
+++ b/bacula/src/qt-console/main.cpp
@@ -226,7 +226,7 @@ static int check_resources()
       tls_needed = director->tls_enable || director->tls_authenticate;
 
       if ((!director->tls_ca_certfile && !director->tls_ca_certdir) && tls_needed) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Director \"%s\" in %s."
                              " At least one CA certificate store is required.\n"),
                              director->hdr.name, configfile);
@@ -256,7 +256,7 @@ static int check_resources()
       tls_needed = cons->tls_enable || cons->tls_authenticate;
 
       if ((!cons->tls_ca_certfile && !cons->tls_ca_certdir) && tls_needed) {
-         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Emsg2(M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
                              " or \"TLS CA Certificate Dir\" are defined for Console \"%s\" in %s.\n"),
                              cons->hdr.name, configfile);
          ok = false;

diff --git a/bacula/src/stored/bsdjson.c b/bacula/src/stored/bsdjson.c
index 457052218562d52997ce2268c9cef123a02d61e0..a61599f981444a76f6ce73fa9dcde288ddaf7815 100644 (file)
--- a/bacula/src/stored/bsdjson.c
+++ b/bacula/src/stored/bsdjson.c
@@ -635,7 +635,7 @@ static int check_resources()
       }
 
       if ((!store->tls_ca_certfile && !store->tls_ca_certdir) && tls_needed && store->tls_verify_peer) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
               " or \"TLS CA Certificate Dir\" are defined for Storage \"%s\" in %s."
               " At least one CA certificate store is required"
               " when using \"TLS Verify Peer\".\n"),
@@ -667,7 +667,7 @@ static int check_resources()
       }
 
       if ((!director->tls_ca_certfile && !director->tls_ca_certdir) && tls_needed && director->tls_verify_peer) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
               " or \"TLS CA Certificate Dir\" are defined for Director \"%s\" in %s."
               " At least one CA certificate store is required"
               " when using \"TLS Verify Peer\".\n"),

diff --git a/bacula/src/stored/stored.c b/bacula/src/stored/stored.c
index 1b185e764b8c7db9754311720250aeefe249b899..fb95c6727e68120a961bae9ca16c428ba1729371 100644 (file)
--- a/bacula/src/stored/stored.c
+++ b/bacula/src/stored/stored.c
@@ -431,7 +431,7 @@ static int check_resources()
       }
 
       if ((!store->tls_ca_certfile && !store->tls_ca_certdir) && tls_needed && store->tls_verify_peer) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
               " or \"TLS CA Certificate Dir\" are defined for Storage \"%s\" in %s."
               " At least one CA certificate store is required"
               " when using \"TLS Verify Peer\".\n"),
@@ -489,7 +489,7 @@ static int check_resources()
       }
 
       if ((!director->tls_ca_certfile && !director->tls_ca_certdir) && tls_needed && director->tls_verify_peer) {
-         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate\""
+         Jmsg(NULL, M_FATAL, 0, _("Neither \"TLS CA Certificate File\""
               " or \"TLS CA Certificate Dir\" are defined for Director \"%s\" in %s."
               " At least one CA certificate store is required"
               " when using \"TLS Verify Peer\".\n"),