Fix calls to sl.set_string()

diff --git a/bacula/src/dird/ua_cmds.c b/bacula/src/dird/ua_cmds.c
index f77fef73e17a513f231b829b57228ae6cbe8451f..f13f090081b8b18d000a6ce79e2ad47ff7fde078 100644 (file)
--- a/bacula/src/dird/ua_cmds.c
+++ b/bacula/src/dird/ua_cmds.c
@@ -1454,7 +1454,7 @@ static void delete_job(UAContext *ua)
 
    i = find_arg_with_value(ua, NT_("jobid"));
    if (i >= 0) {
-      if (sl.set_string(ua->argv[i], true) < 0 && sl.get_errmsg()) {
+      if (!sl.set_string(ua->argv[i], true)) {
          ua->warning_msg("%s", sl.get_errmsg());
          return;
       }

diff --git a/bacula/src/dird/ua_input.c b/bacula/src/dird/ua_input.c
index d8dd4aff1e75b01b7701030600b053fc9dee59c5..421a9e620056b8640dc8ddaa693c168f242ccd6b 100644 (file)
--- a/bacula/src/dird/ua_input.c
+++ b/bacula/src/dird/ua_input.c
@@ -86,7 +86,7 @@ bool get_selection_list(UAContext *ua, sellist &sl,
       if (!get_cmd(ua, prompt, subprompt)) {
          return false;
       }
-      if (sl.set_string(ua->cmd, true) < 0 && sl.get_errmsg()) {
+      if (!sl.set_string(ua->cmd, true)) {
          ua->send_msg("%s", sl.get_errmsg());
          continue;
       }

diff --git a/bacula/src/dird/ua_run.c b/bacula/src/dird/ua_run.c
index 45a925879f112a6145628898e07b96c0ea85f567..4a31ac66d3be237c43f8872e57f76136a3032739 100644 (file)
--- a/bacula/src/dird/ua_run.c
+++ b/bacula/src/dird/ua_run.c
@@ -419,7 +419,7 @@ static bool get_jobid_list(UAContext *ua, sellist &sl, run_ctx &rc)
          ua->send_msg(_("No JobId specified.\n"));
          return false;
       }
-      if (sl.set_string(ua->argv[i], true) < 0 && sl.get_errmsg()) {
+      if (!sl.set_string(ua->argv[i], true)) {
          ua->send_msg("%s", sl.get_errmsg());
          return false;
       }

diff --git a/bacula/src/dird/ua_select.c b/bacula/src/dird/ua_select.c
index 66b27205b0335acf1cfe000f620eefc07e90e6af..68904abe593083b36a8d186827b3ade878b8b4b1 100644 (file)
--- a/bacula/src/dird/ua_select.c
+++ b/bacula/src/dird/ua_select.c
@@ -1241,7 +1241,7 @@ int select_running_jobs(UAContext *ua, alist *jcrs, const char *reason)
          sellist sl;
          int32_t JobId;
 
-         if (sl.set_string(ua->argv[i], true) < 0 && sl.get_errmsg()) {
+         if (!sl.set_string(ua->argv[i], true)) {
             ua->send_msg("%s", sl.get_errmsg());
             break;
          }