]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/dird/ua_status.c
This commit was manufactured by cvs2svn to create tag
[bacula/bacula] / bacula / src / dird / ua_status.c
index 4e79cc230bf1fd779998d4e6ab4b2aecb724b472..16b9581cf60f6e4e8a8071c0d44a13a93e9ad3b6 100644 (file)
@@ -6,27 +6,22 @@
  *
  *   Version $Id$
  */
-
 /*
-   Copyright (C) 2000-2003 Kern Sibbald and John Walker
+   Copyright (C) 2001-2006 Kern Sibbald
 
    This program is free software; you can redistribute it and/or
-   modify it under the terms of the GNU General Public License as
-   published by the Free Software Foundation; either version 2 of
-   the License, or (at your option) any later version.
+   modify it under the terms of the GNU General Public License
+   version 2 as amended with additional clauses defined in the
+   file LICENSE in the main source directory.
 
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
-   General Public License for more details.
-
-   You should have received a copy of the GNU General Public
-   License along with this program; if not, write to the Free
-   Software Foundation, Inc., 59 Temple Place - Suite 330, Boston,
-   MA 02111-1307, USA.
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 
+   the file LICENSE for additional details.
 
  */
 
+
 #include "bacula.h"
 #include "dird.h"
 
@@ -39,13 +34,59 @@ static void list_running_jobs(UAContext *ua);
 static void list_terminated_jobs(UAContext *ua);
 static void do_storage_status(UAContext *ua, STORE *store);
 static void do_client_status(UAContext *ua, CLIENT *client);
-static void do_director_status(UAContext *ua, char *cmd);
-static void do_all_status(UAContext *ua, char *cmd);
+static void do_director_status(UAContext *ua);
+static void do_all_status(UAContext *ua);
+
+static char OKqstatus[]   = "1000 OK .status\n";
+static char DotStatusJob[] = "JobId=%s JobStatus=%c JobErrors=%d\n";
+
+/*
+ * .status command
+ */
+int qstatus_cmd(UAContext *ua, const char *cmd)
+{
+   JCR* njcr = NULL;
+   s_last_job* job;
+   char ed1[50];
+
+   if (!open_db(ua)) {
+      return 1;
+   }
+   Dmsg1(20, "status:%s:\n", cmd);
+
+   if ((ua->argc != 3) || (strcasecmp(ua->argk[1], "dir"))) {
+      bsendmsg(ua, "1900 Bad .status command, missing arguments.\n");
+      return 1;
+   }
+
+   if (strcasecmp(ua->argk[2], "current") == 0) {
+      bsendmsg(ua, OKqstatus, ua->argk[2]);
+      foreach_jcr(njcr) {
+         if (njcr->JobId != 0) {
+            bsendmsg(ua, DotStatusJob, edit_int64(njcr->JobId, ed1), 
+                     njcr->JobStatus, njcr->JobErrors);
+         }
+      }
+      endeach_jcr(njcr);
+   } else if (strcasecmp(ua->argk[2], "last") == 0) {
+      bsendmsg(ua, OKqstatus, ua->argk[2]);
+      if ((last_jobs) && (last_jobs->size() > 0)) {
+         job = (s_last_job*)last_jobs->last();
+         bsendmsg(ua, DotStatusJob, edit_int64(job->JobId, ed1), 
+                  job->JobStatus, job->Errors);
+      }
+   } else {
+      bsendmsg(ua, "1900 Bad .status command, wrong argument.\n");
+      return 1;
+   }
+
+   return 1;
+}
 
 /*
  * status command
  */
-int status_cmd(UAContext *ua, char *cmd)
+int status_cmd(UAContext *ua, const char *cmd)
 {
    STORE *store;
    CLIENT *client;
@@ -57,73 +98,75 @@ int status_cmd(UAContext *ua, char *cmd)
    Dmsg1(20, "status:%s:\n", cmd);
 
    for (i=1; i<ua->argc; i++) {
-      if (strcasecmp(ua->argk[i], _("all")) == 0) {
-        do_all_status(ua, cmd);
-        return 1;
-      } else if (strcasecmp(ua->argk[i], _("dir")) == 0 ||
-                 strcasecmp(ua->argk[i], _("director")) == 0) {
-        do_director_status(ua, cmd);
-        return 1;
-      } else if (strcasecmp(ua->argk[i], _("client")) == 0) {
-        client = get_client_resource(ua);
-        if (client) {
-           do_client_status(ua, client);
-        }
-        return 1;
+      if (strcasecmp(ua->argk[i], N_("all")) == 0) {
+         do_all_status(ua);
+         return 1;
+      } else if (strcasecmp(ua->argk[i], N_("dir")) == 0 ||
+                 strcasecmp(ua->argk[i], N_("director")) == 0) {
+         do_director_status(ua);
+         return 1;
+      } else if (strcasecmp(ua->argk[i], N_("client")) == 0) {
+         client = get_client_resource(ua);
+         if (client) {
+            do_client_status(ua, client);
+         }
+         return 1;
       } else {
-        store = get_storage_resource(ua, 0);
-        if (store) {
-           do_storage_status(ua, store);
-        }
-        return 1;
+         store = get_storage_resource(ua, false/*no default*/);
+         if (store) {
+            do_storage_status(ua, store);
+         }
+         return 1;
       }
    }
    /* If no args, ask for status type */
-   if (ua->argc == 1) {                                   
+   if (ua->argc == 1) {
+       char prmt[MAX_NAME_LENGTH];
+
       start_prompt(ua, _("Status available for:\n"));
-      add_prompt(ua, _("Director"));
-      add_prompt(ua, _("Storage"));
-      add_prompt(ua, _("Client"));
-      add_prompt(ua, _("All"));
+      add_prompt(ua, N_("Director"));
+      add_prompt(ua, N_("Storage"));
+      add_prompt(ua, N_("Client"));
+      add_prompt(ua, N_("All"));
       Dmsg0(20, "do_prompt: select daemon\n");
-      if ((item=do_prompt(ua, "",  _("Select daemon type for status"), cmd, MAX_NAME_LENGTH)) < 0) {
-        return 1;
+      if ((item=do_prompt(ua, "",  _("Select daemon type for status"), prmt, sizeof(prmt))) < 0) {
+         return 1;
       }
       Dmsg1(20, "item=%d\n", item);
-      switch (item) { 
-      case 0:                        /* Director */
-        do_director_status(ua, cmd);
-        break;
+      switch (item) {
+      case 0:                         /* Director */
+         do_director_status(ua);
+         break;
       case 1:
-        store = select_storage_resource(ua);
-        if (store) {
-           do_storage_status(ua, store);
-        }
-        break;
+         store = select_storage_resource(ua);
+         if (store) {
+            do_storage_status(ua, store);
+         }
+         break;
       case 2:
-        client = select_client_resource(ua);
-        if (client) {
-           do_client_status(ua, client);
-        }
-        break;
+         client = select_client_resource(ua);
+         if (client) {
+            do_client_status(ua, client);
+         }
+         break;
       case 3:
-        do_all_status(ua, cmd);
-        break;
+         do_all_status(ua);
+         break;
       default:
-        break;
+         break;
       }
    }
    return 1;
 }
 
-static void do_all_status(UAContext *ua, char *cmd)
+static void do_all_status(UAContext *ua)
 {
    STORE *store, **unique_store;
    CLIENT *client, **unique_client;
    int i, j;
    bool found;
 
-   do_director_status(ua, cmd);
+   do_director_status(ua);
 
    /* Count Storage items */
    LockRes();
@@ -132,22 +175,22 @@ static void do_all_status(UAContext *ua, char *cmd)
       i++;
    }
    unique_store = (STORE **) malloc(i * sizeof(STORE));
-   /* Find Unique Storage address/port */        
+   /* Find Unique Storage address/port */
    i = 0;
    foreach_res(store, R_STORAGE) {
       found = false;
       if (!acl_access_ok(ua, Storage_ACL, store->hdr.name)) {
-        continue;
+         continue;
       }
       for (j=0; j<i; j++) {
-        if (strcmp(unique_store[j]->address, store->address) == 0 &&
-            unique_store[j]->SDport == store->SDport) {
-           found = true;
-           break;
-        }
+         if (strcmp(unique_store[j]->address, store->address) == 0 &&
+             unique_store[j]->SDport == store->SDport) {
+            found = true;
+            break;
+         }
       }
       if (!found) {
-        unique_store[i++] = store;
+         unique_store[i++] = store;
          Dmsg2(40, "Stuffing: %s:%d\n", store->address, store->SDport);
       }
    }
@@ -166,22 +209,22 @@ static void do_all_status(UAContext *ua, char *cmd)
       i++;
    }
    unique_client = (CLIENT **)malloc(i * sizeof(CLIENT));
-   /* Find Unique Client address/port */        
+   /* Find Unique Client address/port */
    i = 0;
    foreach_res(client, R_CLIENT) {
       found = false;
       if (!acl_access_ok(ua, Client_ACL, client->hdr.name)) {
-        continue;
+         continue;
       }
       for (j=0; j<i; j++) {
-        if (strcmp(unique_client[j]->address, client->address) == 0 &&
-            unique_client[j]->FDport == client->FDport) {
-           found = true;
-           break;
-        }
+         if (strcmp(unique_client[j]->address, client->address) == 0 &&
+             unique_client[j]->FDport == client->FDport) {
+            found = true;
+            break;
+         }
       }
       if (!found) {
-        unique_client[i++] = client;
+         unique_client[i++] = client;
          Dmsg2(40, "Stuffing: %s:%d\n", client->address, client->FDport);
       }
    }
@@ -192,50 +235,63 @@ static void do_all_status(UAContext *ua, char *cmd)
       do_client_status(ua, unique_client[j]);
    }
    free(unique_client);
-   
+
 }
 
-static void do_director_status(UAContext *ua, char *cmd)
+static void do_director_status(UAContext *ua)
 {
    char dt[MAX_TIME_LENGTH];
 
-   bsendmsg(ua, "%s Version: " VERSION " (" BDATE ") %s %s %s\n", my_name,
-           HOST_OS, DISTNAME, DISTVER);
+   bsendmsg(ua, _("%s Version: %s (%s) %s %s %s\n"), my_name, VERSION, BDATE,
+            HOST_OS, DISTNAME, DISTVER);
    bstrftime_nc(dt, sizeof(dt), daemon_start_time);
-   bsendmsg(ua, _("Daemon started %s, %d Job%s run since started.\n"), 
-        dt, num_jobs_run, num_jobs_run == 1 ? "" : "s");
+   if (num_jobs_run == 1) {
+      bsendmsg(ua, _("Daemon started %s, 1 Job run since started.\n"), dt);
+   }
+   else {
+      bsendmsg(ua, _("Daemon started %s, %d Jobs run since started.\n"),
+        dt, num_jobs_run);
+   }
+   if (debug_level > 0) {
+      char b1[35], b2[35], b3[35], b4[35];
+      bsendmsg(ua, _(" Heap: bytes=%s max_bytes=%s bufs=%s max_bufs=%s\n"),
+            edit_uint64_with_commas(sm_bytes, b1),
+            edit_uint64_with_commas(sm_max_bytes, b2),
+            edit_uint64_with_commas(sm_buffers, b3),
+            edit_uint64_with_commas(sm_max_buffers, b4));
+   }
    /*
     * List scheduled Jobs
     */
    list_scheduled_jobs(ua);
 
-   /* 
+   /*
     * List running jobs
     */
    list_running_jobs(ua);
 
-   /* 
+   /*
     * List terminated jobs
     */
    list_terminated_jobs(ua);
-   bsendmsg(ua, "====\n");
+   bsendmsg(ua, _("====\n"));
 }
 
 static void do_storage_status(UAContext *ua, STORE *store)
 {
    BSOCK *sd;
 
-   ua->jcr->store = store;
+   set_storage(ua->jcr, store);
    /* Try connecting for up to 15 seconds */
-   bsendmsg(ua, _("Connecting to Storage daemon %s at %s:%d\n"), 
+   bsendmsg(ua, _("Connecting to Storage daemon %s at %s:%d\n"),
       store->hdr.name, store->address, store->SDport);
    if (!connect_to_storage_daemon(ua->jcr, 1, 15, 0)) {
       bsendmsg(ua, _("\nFailed to connect to Storage daemon %s.\n====\n"),
-        store->hdr.name);
+         store->hdr.name);
       if (ua->jcr->store_bsock) {
-        bnet_close(ua->jcr->store_bsock);
-        ua->jcr->store_bsock = NULL;
-      }        
+         bnet_close(ua->jcr->store_bsock);
+         ua->jcr->store_bsock = NULL;
+      }
       return;
    }
    Dmsg0(20, _("Connected to storage daemon\n"));
@@ -247,9 +303,9 @@ static void do_storage_status(UAContext *ua, STORE *store)
    bnet_sig(sd, BNET_TERMINATE);
    bnet_close(sd);
    ua->jcr->store_bsock = NULL;
-   return;  
+   return;
 }
-   
+
 static void do_client_status(UAContext *ua, CLIENT *client)
 {
    BSOCK *fd;
@@ -265,15 +321,15 @@ static void do_client_status(UAContext *ua, CLIENT *client)
    ua->jcr->sd_auth_key = bstrdup("dummy");
 
    /* Try to connect for 15 seconds */
-   bsendmsg(ua, _("Connecting to Client %s at %s:%d\n"), 
+   bsendmsg(ua, _("Connecting to Client %s at %s:%d\n"),
       client->hdr.name, client->address, client->FDport);
    if (!connect_to_file_daemon(ua->jcr, 1, 15, 0)) {
       bsendmsg(ua, _("Failed to connect to Client %s.\n====\n"),
-        client->hdr.name);
+         client->hdr.name);
       if (ua->jcr->file_bsock) {
-        bnet_close(ua->jcr->file_bsock);
-        ua->jcr->file_bsock = NULL;
-      }        
+         bnet_close(ua->jcr->file_bsock);
+         ua->jcr->file_bsock = NULL;
+      }
       return;
    }
    Dmsg0(20, _("Connected to file daemon\n"));
@@ -286,58 +342,93 @@ static void do_client_status(UAContext *ua, CLIENT *client)
    bnet_close(fd);
    ua->jcr->file_bsock = NULL;
 
-   return;  
+   return;
 }
 
 static void prt_runhdr(UAContext *ua)
 {
    bsendmsg(ua, _("\nScheduled Jobs:\n"));
-   bsendmsg(ua, _("Level          Type     Scheduled          Name               Volume\n"));
-   bsendmsg(ua, _("===============================================================================\n"));
+   bsendmsg(ua, _("Level          Type     Pri  Scheduled          Name               Volume\n"));
+   bsendmsg(ua, _("===================================================================================\n"));
 }
 
-static void prt_runtime(UAContext *ua, JOB *job, int level, time_t runtime, POOL *pool)
+/* Scheduling packet */
+struct sched_pkt {
+   dlink link;                        /* keep this as first item!!! */
+   JOB *job;
+   int level;
+   int priority;
+   time_t runtime;
+   POOL *pool;
+   STORE *store;
+};
+
+static void prt_runtime(UAContext *ua, sched_pkt *sp)
 {
-   char dt[MAX_TIME_LENGTH];      
-   char *level_ptr;
+   char dt[MAX_TIME_LENGTH];
+   const char *level_ptr;
    bool ok = false;
    bool close_db = false;
    JCR *jcr = ua->jcr;
    MEDIA_DBR mr;
+
    memset(&mr, 0, sizeof(mr));
-   if (job->JobType == JT_BACKUP) {
+   if (sp->job->JobType == JT_BACKUP) {
       jcr->db = NULL;
-      ok = complete_jcr_for_job(jcr, job, pool);
+      ok = complete_jcr_for_job(jcr, sp->job, sp->pool);
       if (jcr->db) {
-        close_db = true;             /* new db opened, remember to close it */
+         close_db = true;             /* new db opened, remember to close it */
       }
       if (ok) {
-        ok = find_next_volume_for_append(jcr, &mr, 0);
+         mr.PoolId = jcr->PoolId;
+         mr.StorageId = sp->store->StorageId;
+         ok = find_next_volume_for_append(jcr, &mr, 1, false/*no create*/);
       }
       if (!ok) {
          bstrncpy(mr.VolumeName, "*unknown*", sizeof(mr.VolumeName));
       }
    }
-   bstrftime_nc(dt, sizeof(dt), runtime);
-   switch (job->JobType) {
+   bstrftime_nc(dt, sizeof(dt), sp->runtime);
+   switch (sp->job->JobType) {
    case JT_ADMIN:
    case JT_RESTORE:
       level_ptr = " ";
       break;
    default:
-      level_ptr = level_to_str(level);
+      level_ptr = level_to_str(sp->level);
       break;
    }
-   bsendmsg(ua, _("%-14s %-8s %-18s %-18s %s\n"), 
-      level_ptr, job_type_to_str(job->JobType), dt, job->hdr.name, mr.VolumeName);
+   bsendmsg(ua, _("%-14s %-8s %3d  %-18s %-18s %s\n"),
+      level_ptr, job_type_to_str(sp->job->JobType), sp->priority, dt,
+      sp->job->hdr.name, mr.VolumeName);
    if (close_db) {
       db_close_database(jcr, jcr->db);
    }
-   jcr->db = ua->db;                 /* restore ua db to jcr */
+   jcr->db = ua->db;                  /* restore ua db to jcr */
 
 }
 
-/*         
+/*
+ * Sort items by runtime, priority
+ */
+static int my_compare(void *item1, void *item2)
+{
+   sched_pkt *p1 = (sched_pkt *)item1;
+   sched_pkt *p2 = (sched_pkt *)item2;
+   if (p1->runtime < p2->runtime) {
+      return -1;
+   } else if (p1->runtime > p2->runtime) {
+      return 1;
+   }
+   if (p1->priority < p2->priority) {
+      return -1;
+   } else if (p1->priority > p2->priority) {
+      return 1;
+   }
+   return 0;
+}
+
+/*
  * Find all jobs to be run in roughly the
  *  next 24 hours.
  */
@@ -346,36 +437,69 @@ static void list_scheduled_jobs(UAContext *ua)
    time_t runtime;
    RUN *run;
    JOB *job;
+   STORE* store;
    int level, num_jobs = 0;
+   int priority;
    bool hdr_printed = false;
+   dlist sched;
+   sched_pkt *sp;
+   int days, i;
 
    Dmsg0(200, "enter list_sched_jobs()\n");
 
+   days = 1;
+   i = find_arg_with_value(ua, N_("days"));
+   if (i >= 0) {
+     days = atoi(ua->argv[i]);
+     if ((days < 0) || (days > 50)) {
+       bsendmsg(ua, _("Ignoring illegal value for days.\n"));
+       days = 1;
+     }
+   }
+
    /* Loop through all jobs */
    LockRes();
    foreach_res(job, R_JOB) {
-      if (!acl_access_ok(ua, Job_ACL, job->hdr.name)) {
-        continue;
+      if (!acl_access_ok(ua, Job_ACL, job->hdr.name) || !job->enabled) {
+         continue;
       }
-      for (run=NULL; (run = find_next_run(run, job, runtime)); ) {
-        level = job->level;   
-        if (run->level) {
-           level = run->level;
-        }
-        if (!hdr_printed) {
-           prt_runhdr(ua);
-           hdr_printed = true;
-        }
-        prt_runtime(ua, job, level, runtime, run->pool);
-        num_jobs++;
+      for (run=NULL; (run = find_next_run(run, job, runtime, days)); ) {
+         level = job->JobLevel;
+         if (run->level) {
+            level = run->level;
+         }
+         priority = job->Priority;
+         if (run->Priority) {
+            priority = run->Priority;
+         }
+         if (run->storage) {
+            store = run->storage;
+         } else {
+            store = (STORE *)job->storage->first();
+         }
+         if (!hdr_printed) {
+            prt_runhdr(ua);
+            hdr_printed = true;
+         }
+         sp = (sched_pkt *)malloc(sizeof(sched_pkt));
+         sp->job = job;
+         sp->level = level;
+         sp->priority = priority;
+         sp->runtime = runtime;
+         sp->pool = run->pool;
+         sp->store = store;
+         sched.binary_insert_multiple(sp, my_compare);
+         num_jobs++;
       }
-
    } /* end for loop over resources */
    UnlockRes();
+   foreach_dlist(sp, &sched) {
+      prt_runtime(ua, sp);
+   }
    if (num_jobs == 0) {
       bsendmsg(ua, _("No Scheduled Jobs.\n"));
-   } 
-   bsendmsg(ua, "====\n");
+   }
+   bsendmsg(ua, _("====\n"));
    Dmsg0(200, "Leave list_sched_jobs_runs()\n");
 }
 
@@ -383,30 +507,30 @@ static void list_running_jobs(UAContext *ua)
 {
    JCR *jcr;
    int njobs = 0;
-   char *msg;
+   const char *msg;
+   char *emsg;                        /* edited message */
    char dt[MAX_TIME_LENGTH];
    char level[10];
    bool pool_mem = false;
 
    Dmsg0(200, "enter list_run_jobs()\n");
-   bsendmsg(ua, _("Running Jobs:\n"));
-   lock_jcr_chain();
+   bsendmsg(ua, _("\nRunning Jobs:\n"));
    foreach_jcr(jcr) {
-      njobs++;
-      if (jcr->JobId == 0) {     /* this is us */
-        /* this is a console or other control job. We only show console
-         * jobs in the status output.
-         */
-        if (jcr->JobType == JT_CONSOLE) {
-           bstrftime_nc(dt, sizeof(dt), jcr->start_time);
+      if (jcr->JobId == 0) {      /* this is us */
+         /* this is a console or other control job. We only show console
+          * jobs in the status output.
+          */
+         if (jcr->JobType == JT_CONSOLE) {
+            bstrftime_nc(dt, sizeof(dt), jcr->start_time);
             bsendmsg(ua, _("Console connected at %s\n"), dt);
-        }
-        njobs--;
-      }
-      free_locked_jcr(jcr);
+         }
+         continue;
+      }       
+      njobs++;
    }
+   endeach_jcr(jcr);
+
    if (njobs == 0) {
-      unlock_jcr_chain();
       /* Note the following message is used in regress -- don't change */
       bsendmsg(ua, _("No Jobs running.\n====\n"));
       Dmsg0(200, "leave list_run_jobs()\n");
@@ -417,125 +541,127 @@ static void list_running_jobs(UAContext *ua)
    bsendmsg(ua, _("======================================================================\n"));
    foreach_jcr(jcr) {
       if (jcr->JobId == 0 || !acl_access_ok(ua, Job_ACL, jcr->job->hdr.name)) {
-        free_locked_jcr(jcr);
-        continue;
+         continue;
       }
       njobs++;
       switch (jcr->JobStatus) {
       case JS_Created:
          msg = _("is waiting execution");
-        break;
+         break;
       case JS_Running:
          msg = _("is running");
-        break;
+         break;
       case JS_Blocked:
          msg = _("is blocked");
-        break;
+         break;
       case JS_Terminated:
          msg = _("has terminated");
-        break;
+         break;
       case JS_ErrorTerminated:
          msg = _("has erred");
-        break;
+         break;
       case JS_Error:
          msg = _("has errors");
-        break;
+         break;
       case JS_FatalError:
          msg = _("has a fatal error");
-        break;
+         break;
       case JS_Differences:
          msg = _("has verify differences");
-        break;
+         break;
       case JS_Canceled:
          msg = _("has been canceled");
-        break;
+         break;
       case JS_WaitFD:
-        msg = (char *) get_pool_memory(PM_FNAME);
-         Mmsg(&msg, _("is waiting on Client %s"), jcr->client->hdr.name);
-        pool_mem = true;
-        break;
+         emsg = (char *) get_pool_memory(PM_FNAME);
+         Mmsg(emsg, _("is waiting on Client %s"), jcr->client->hdr.name);
+         pool_mem = true;
+         msg = emsg;
+         break;
       case JS_WaitSD:
-        msg = (char *) get_pool_memory(PM_FNAME);
-         Mmsg(&msg, _("is waiting on Storage %s"), jcr->store->hdr.name);
-        pool_mem = true;
-        break;
+         emsg = (char *) get_pool_memory(PM_FNAME);
+         Mmsg(emsg, _("is waiting on Storage %s"), jcr->store->hdr.name);
+         pool_mem = true;
+         msg = emsg;
+         break;
       case JS_WaitStoreRes:
          msg = _("is waiting on max Storage jobs");
-        break;
+         break;
       case JS_WaitClientRes:
          msg = _("is waiting on max Client jobs");
-        break;
+         break;
       case JS_WaitJobRes:
          msg = _("is waiting on max Job jobs");
-        break;
+         break;
       case JS_WaitMaxJobs:
          msg = _("is waiting on max total jobs");
-        break;
+         break;
       case JS_WaitStartTime:
          msg = _("is waiting for its start time");
-        break;
+         break;
       case JS_WaitPriority:
          msg = _("is waiting for higher priority jobs to finish");
-        break;
+         break;
 
       default:
-        msg = (char *) get_pool_memory(PM_FNAME);
-         Mmsg(&msg, _("is in unknown state %c"), jcr->JobStatus);
-        pool_mem = true;
-        break;
+         emsg = (char *) get_pool_memory(PM_FNAME);
+         Mmsg(emsg, _("is in unknown state %c"), jcr->JobStatus);
+         pool_mem = true;
+         msg = emsg;
+         break;
       }
-      /* 
-       * Now report Storage daemon status code 
+      /*
+       * Now report Storage daemon status code
        */
       switch (jcr->SDJobStatus) {
       case JS_WaitMount:
-        if (pool_mem) {
-           free_pool_memory(msg);
-           pool_mem = false;
-        }
+         if (pool_mem) {
+            free_pool_memory(emsg);
+            pool_mem = false;
+         }
          msg = _("is waiting for a mount request");
-        break;
+         break;
       case JS_WaitMedia:
-        if (pool_mem) {
-           free_pool_memory(msg);
-           pool_mem = false;
-        }
+         if (pool_mem) {
+            free_pool_memory(emsg);
+            pool_mem = false;
+         }
          msg = _("is waiting for an appendable Volume");
-        break;
+         break;
       case JS_WaitFD:
-        if (!pool_mem) {
-           msg = (char *) get_pool_memory(PM_FNAME);
-           pool_mem = true;
-        }
-         Mmsg(&msg, _("is waiting for Client %s to connect to Storage %s"),
-             jcr->client->hdr.name, jcr->store->hdr.name);
-        break;
+         if (!pool_mem) {
+            emsg = (char *) get_pool_memory(PM_FNAME);
+            pool_mem = true;
+         }
+         Mmsg(emsg, _("is waiting for Client %s to connect to Storage %s"),
+              jcr->client->hdr.name, jcr->store->hdr.name);
+         msg = emsg;
+         break;
       }
       switch (jcr->JobType) {
       case JT_ADMIN:
       case JT_RESTORE:
          bstrncpy(level, "      ", sizeof(level));
-        break;
+         break;
       default:
-        bstrncpy(level, level_to_str(jcr->JobLevel), sizeof(level));
-        level[7] = 0;
-        break;
+         bstrncpy(level, level_to_str(jcr->JobLevel), sizeof(level));
+         level[7] = 0;
+         break;
       }
 
-      bsendmsg(ua, _("%6d %-6s  %-20s %s\n"), 
-        jcr->JobId,
-        level, 
-        jcr->Job,
-        msg);
+      bsendmsg(ua, _("%6d %-6s  %-20s %s\n"),
+         jcr->JobId,
+         level,
+         jcr->Job,
+         msg);
 
       if (pool_mem) {
-        free_pool_memory(msg);
-        pool_mem = false;
+         free_pool_memory(emsg);
+         pool_mem = false;
       }
-      free_locked_jcr(jcr);
    }
-   unlock_jcr_chain();
-   bsendmsg(ua, "====\n");
+   endeach_jcr(jcr);
+   bsendmsg(ua, _("====\n"));
    Dmsg0(200, "leave list_run_jobs()\n");
 }
 
@@ -555,56 +681,61 @@ static void list_terminated_jobs(UAContext *ua)
    bsendmsg(ua, _("========================================================================\n"));
    foreach_dlist(je, last_jobs) {
       char JobName[MAX_NAME_LENGTH];
-      char *termstat;
+      const char *termstat;
+
+      bstrncpy(JobName, je->Job, sizeof(JobName));
+      /* There are three periods after the Job name */
+      char *p;
+      for (int i=0; i<3; i++) {
+         if ((p=strrchr(JobName, '.')) != NULL) {
+            *p = 0;
+         }
+      }
+
+      if (!acl_access_ok(ua, Job_ACL, JobName)) {
+         continue;
+      }
 
       bstrftime_nc(dt, sizeof(dt), je->end_time);
       switch (je->JobType) {
       case JT_ADMIN:
       case JT_RESTORE:
          bstrncpy(level, "    ", sizeof(level));
-        break;
+         break;
       default:
-        bstrncpy(level, level_to_str(je->JobLevel), sizeof(level));
-        level[4] = 0;
-        break;
+         bstrncpy(level, level_to_str(je->JobLevel), sizeof(level));
+         level[4] = 0;
+         break;
       }
       switch (je->JobStatus) {
       case JS_Created:
-         termstat = "Created";
-        break;
+         termstat = _("Created");
+         break;
       case JS_FatalError:
       case JS_ErrorTerminated:
-         termstat = "Error";
-        break;
+         termstat = _("Error");
+         break;
       case JS_Differences:
-         termstat = "Diffs";
-        break;
+         termstat = _("Diffs");
+         break;
       case JS_Canceled:
-         termstat = "Cancel";
-        break;
+         termstat = _("Cancel");
+         break;
       case JS_Terminated:
-         termstat = "OK";
-        break;
+         termstat = _("OK");
+         break;
       default:
-         termstat = "Other";
-        break;
+         termstat = _("Other");
+         break;
       }
-      bstrncpy(JobName, je->Job, sizeof(JobName));
-      /* There are three periods after the Job name */
-      char *p;
-      for (int i=0; i<3; i++) {
-         if ((p=strrchr(JobName, '.')) != NULL) {
-           *p = 0;
-        }
-      }
-      bsendmsg(ua, _("%6d  %-6s %8s %14s %-7s  %-8s %s\n"), 
-        je->JobId,
-        level, 
-        edit_uint64_with_commas(je->JobFiles, b1),
-        edit_uint64_with_commas(je->JobBytes, b2), 
-        termstat,
-        dt, JobName);
-   }
-   bsendmsg(ua, "\n");
+      bsendmsg(ua, _("%6d  %-6s %8s %14s %-7s  %-8s %s\n"),
+         je->JobId,
+         level,
+         edit_uint64_with_commas(je->JobFiles, b1),
+         edit_uint64_with_commas(je->JobBytes, b2),
+         termstat,
+         dt, JobName);
+   }
+   bsendmsg(ua, _("\n"));
    unlock_last_jobs_list();
 }