]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/lib/message.c
Fix conio.h problem on Solaris
[bacula/bacula] / bacula / src / lib / message.c
index 622f5ed0ea26b94e21898195628774142bed5c01..8519389e9f2c1560bc9f60cbf7dc4a0f1e09c004 100755 (executable)
@@ -3,10 +3,12 @@
  *
  *   Kern Sibbald, April 2000 
  *
+ *   Version $Id$
+ *
  */
 
 /*
-   Copyright (C) 2000, 2001, 2002 Kern Sibbald and John Walker
+   Copyright (C) 2000-2004 Kern Sibbald and John Walker
 
    This program is free software; you can redistribute it and/or
    modify it under the terms of the GNU General Public License as
 #include "bacula.h"
 #include "jcr.h"
 
+#if !defined(HAVE_CONSOLE)
+#if defined(HAVE_CYGWIN) || defined(HAVE_WIN32)
+#include <windows.h>
+#endif
+#endif
+
 #define FULL_LOCATION 1              /* set for file:line in Debug messages */
 
-char *working_directory = NULL;       /* working directory path stored here */
-int debug_level = 5;                 /* debug level */
+/* 
+ *  This is where we define "Globals" because all the
+ *    daemons include this file.
+ */
+const char *working_directory = NULL;      /* working directory path stored here */
+int verbose = 0;                     /* increase User messages */
+int debug_level = 0;                 /* debug level */
 time_t daemon_start_time = 0;        /* Daemon start time */
-
-char my_name[20];                    /* daemon name is stored here */
+const char *version = VERSION " (" BDATE ")";
+char my_name[30];                    /* daemon name is stored here */
 char *exepath = (char *)NULL;
 char *exename = (char *)NULL;
 int console_msg_pending = 0;
-char con_fname[1000];
-FILE *con_fd = NULL;
+char con_fname[500];                 /* Console filename */
+FILE *con_fd = NULL;                 /* Console file descriptor */
+brwlock_t con_lock;                  /* Console lock structure */
+
+#ifdef HAVE_POSTGRESQL
+char catalog_db[] = "PostgreSQL";
+#else
+#ifdef HAVE_MYSQL
+char catalog_db[] = "MySQL";
+#else 
+#ifdef HAVE_SQLITE
+char catalog_db[] = "SQLite";
+#else
+char catalog_db[] = "Internal";
+#endif
+#endif
+#endif
+
+const char *host_os = HOST_OS;
+const char *distname = DISTNAME;
+const char *distver = DISTVER;
+static FILE *trace_fd = NULL;
+#ifdef HAVE_WIN32
+static bool trace = true;
+#else
+static bool trace = false;
+#endif
 
 /* Forward referenced functions */
 
 /* Imported functions */
 
-/* This chain contains all the possible destinations */
-DEST *dest_chain = NULL;
-/* 
- * send_msg has a bit set for each type that has a 
- * message destination. The info in send_msg[] is 
- * contained in the dest structures,
- * but we keep it here for speed so that we don't have to
- * search all the structures in all the cases. 
- */
-char send_msg[nbytes_for_bits(M_MAX+1)];
+
+/* Static storage */
+
+static MSGS *daemon_msgs;             /* global messages */
+
+/* Define if e_msg must exit when M_ERROR_TERM is received */
+static int exit_on_error = 1;
 
 /* 
  * Set daemon name. Also, find canonical execution
@@ -68,14 +103,13 @@ char send_msg[nbytes_for_bits(M_MAX+1)];
  *  argv is NULL to avoid doing the path code twice.
  */
 #define BTRACE_EXTRA 20
-void my_name_is(int argc, char *argv[], char *name)
+void my_name_is(int argc, char *argv[], const char *name)
 {
    char *l, *p, *q;
    char cpath[400], npath[400];
    int len;
 
-   strncpy(my_name, name, sizeof(my_name));
-   my_name[sizeof(my_name)-1] = 0;
+   bstrncpy(my_name, name, sizeof(my_name));
    if (argc>0 && argv && argv[0]) {
       /* strip trailing filename and save exepath */
       for (l=p=argv[0]; *p; p++) {
@@ -87,7 +121,7 @@ void my_name_is(int argc, char *argv[], char *name)
         l++;
       } else {
         l = argv[0];
-#ifdef HAVE_CYGWIN
+#if defined(HAVE_CYGWIN) || defined(HAVE_WIN32)
         /* On Windows allow c: junk */
          if (l[1] == ':') {
            l += 2;
@@ -100,6 +134,7 @@ void my_name_is(int argc, char *argv[], char *name)
       }
       exename = (char *)malloc(len);
       strcpy(exename, l);
+
       if (exepath) {
         free(exepath);
       }
@@ -129,44 +164,108 @@ void my_name_is(int argc, char *argv[], char *name)
    }
 }
 
-/* Initialize message handler */
+/* 
+ * Initialize message handler for a daemon or a Job
+ *   We make a copy of the MSGS resource passed, so it belows
+ *   to the job or daemon and thus can be modified.
+ * 
+ *   NULL for jcr -> initialize global messages for daemon
+ *   non-NULL    -> initialize jcr using Message resource
+ */
 void
-init_msg(void *vjcr)
+init_msg(JCR *jcr, MSGS *msg)
 {
    DEST *d, *dnew, *temp_chain = NULL;
-   JCR *jcr = (JCR *)vjcr;
-
-   if (!jcr) { 
-      memset(send_msg, 0, sizeof(send_msg));  /* init daemon stuff */
-   } else {                                  /* init for job */
-      /* Walk down the global chain duplicating it
-       * for the current Job.  No need to duplicate
-       * the attached strings.
-       */
-      for (d=dest_chain; d; d=d->next) {
-        dnew = (DEST *) malloc(sizeof(DEST));
-        memcpy(dnew, d, sizeof(DEST));
-        dnew->next = temp_chain;
-        dnew->fd = NULL;
-        temp_chain = dnew;
+   int i;
+
+   if (jcr == NULL && msg == NULL) {
+      init_last_jobs_list();
+   }
+
+#ifndef HAVE_WIN32
+   /*
+    * Make sure we have fd's 0, 1, 2 open
+    *  If we don't do this one of our sockets may open
+    *  there and if we then use stdout, it could
+    *  send total garbage to our socket.
+    *
+    */
+   int fd;
+   fd = open("/dev/null", O_RDONLY, 0644);
+   if (fd > 2) {
+      close(fd);
+   } else {
+      for(i=1; fd + i <= 2; i++) {
+        dup2(fd, fd+i);
+      }
+   }
+
+#endif
+   /*
+    * If msg is NULL, initialize global chain for STDOUT and syslog
+    */
+   if (msg == NULL) {
+      daemon_msgs = (MSGS *)malloc(sizeof(MSGS));
+      memset(daemon_msgs, 0, sizeof(MSGS));
+      for (i=1; i<=M_MAX; i++) {
+#ifndef WIN32
+        add_msg_dest(daemon_msgs, MD_STDOUT, i, NULL, NULL);
+#endif
+        add_msg_dest(daemon_msgs, MD_SYSLOG, i, NULL, NULL);
+      }
+      Dmsg1(050, "Create daemon global message resource 0x%x\n", daemon_msgs);
+      return;
+   }
+
+   /*
+    * Walk down the message resource chain duplicating it
+    * for the current Job.
+    */
+   for (d=msg->dest_chain; d; d=d->next) {
+      dnew = (DEST *)malloc(sizeof(DEST));
+      memcpy(dnew, d, sizeof(DEST));
+      dnew->next = temp_chain;
+      dnew->fd = NULL;
+      dnew->mail_filename = NULL;
+      if (d->mail_cmd) {
+        dnew->mail_cmd = bstrdup(d->mail_cmd);
       }
+      if (d->where) {
+        dnew->where = bstrdup(d->where);
+      }
+      temp_chain = dnew;
+   }
 
-      jcr->dest_chain = temp_chain;
-      memcpy(jcr->send_msg, send_msg, sizeof(send_msg));
+   if (jcr) {
+      jcr->jcr_msgs = (MSGS *)malloc(sizeof(MSGS));
+      memset(jcr->jcr_msgs, 0, sizeof(MSGS));
+      jcr->jcr_msgs->dest_chain = temp_chain;
+      memcpy(jcr->jcr_msgs->send_msg, msg->send_msg, sizeof(msg->send_msg));
+   } else {
+      /* If we have default values, release them now */
+      if (daemon_msgs) {
+        free_msgs_res(daemon_msgs);
+      }
+      daemon_msgs = (MSGS *)malloc(sizeof(MSGS));
+      memset(daemon_msgs, 0, sizeof(MSGS));
+      daemon_msgs->dest_chain = temp_chain;
+      memcpy(daemon_msgs->send_msg, msg->send_msg, sizeof(msg->send_msg));
    }
+   Dmsg2(050, "Copy message resource 0x%x to 0x%x\n", msg, temp_chain);
+
 }
 
 /* Initialize so that the console (User Agent) can
  * receive messages -- stored in a file.
  */
-void init_console_msg(char *wd)
+void init_console_msg(const char *wd)
 {
    int fd;
 
-   sprintf(con_fname, "%s/%s.conmsg", wd, my_name);
+   bsnprintf(con_fname, sizeof(con_fname), "%s/%s.conmsg", wd, my_name);
    fd = open(con_fname, O_CREAT|O_RDWR|O_BINARY, 0600);
    if (fd == -1) {
-       Emsg2(M_ABORT, 0, "Could not open console message file %s: ERR=%s\n",
+      Emsg2(M_ERROR_TERM, 0, _("Could not open console message file %s: ERR=%s\n"),
          con_fname, strerror(errno));
    }
    if (lseek(fd, 0, SEEK_END) > 0) {
@@ -175,9 +274,13 @@ void init_console_msg(char *wd)
    close(fd);
    con_fd = fopen(con_fname, "a+");
    if (!con_fd) {
-       Emsg2(M_ERROR, 0, "Could not open console message file %s: ERR=%s\n",
+      Emsg2(M_ERROR, 0, _("Could not open console message file %s: ERR=%s\n"),
          con_fname, strerror(errno));
    }
+   if (rwl_init(&con_lock) != 0) {
+      Emsg1(M_ERROR_TERM, 0, _("Could not get con mutex: ERR=%s\n"), 
+        strerror(errno));
+   }
 }
 
 /* 
@@ -189,30 +292,30 @@ void init_console_msg(char *wd)
  *  but in the case of MAIL is a space separated list of
  *  email addresses, ...
  */
-void add_msg_dest(int dest_code, int msg_type, char *where, char *mail_cmd)
+void add_msg_dest(MSGS *msg, int dest_code, int msg_type, char *where, char *mail_cmd)
 {
    DEST *d; 
-
-   /* First search the existing chain and see if we
+   /*
+    * First search the existing chain and see if we
     * can simply add this msg_type to an existing entry.
     */
-   for (d=dest_chain; d; d=d->next) {
+   for (d=msg->dest_chain; d; d=d->next) {
       if (dest_code == d->dest_code && ((where == NULL && d->where == NULL) ||
                     (strcmp(where, d->where) == 0))) {  
          Dmsg4(200, "Add to existing d=%x msgtype=%d destcode=%d where=%s\n", 
-            d, msg_type, dest_code, where);
+            d, msg_type, dest_code, NPRT(where));
         set_bit(msg_type, d->msg_types);
-        set_bit(msg_type, send_msg);  /* set msg_type bit in our local */
+        set_bit(msg_type, msg->send_msg);  /* set msg_type bit in our local */
         return;
       }
    }
    /* Not found, create a new entry */
-   d = (DEST *) malloc(sizeof(DEST));
+   d = (DEST *)malloc(sizeof(DEST));
    memset(d, 0, sizeof(DEST));
-   d->next = dest_chain;
+   d->next = msg->dest_chain;
    d->dest_code = dest_code;
    set_bit(msg_type, d->msg_types);     /* set type bit in structure */
-   set_bit(msg_type, send_msg);         /* set type bit in our local */
+   set_bit(msg_type, msg->send_msg);    /* set type bit in our local */
    if (where) {
       d->where = bstrdup(where);
    }
@@ -220,9 +323,8 @@ void add_msg_dest(int dest_code, int msg_type, char *where, char *mail_cmd)
       d->mail_cmd = bstrdup(mail_cmd);
    }
    Dmsg5(200, "add new d=%x msgtype=%d destcode=%d where=%s mailcmd=%s\n", 
-          d, msg_type, dest_code, where?where:"(null)", 
-          d->mail_cmd?d->mail_cmd:"(null)");
-   dest_chain = d;
+         d, msg_type, dest_code, NPRT(where), NPRT(d->mail_cmd));
+   msg->dest_chain = d;
 }
 
 /* 
@@ -230,12 +332,12 @@ void add_msg_dest(int dest_code, int msg_type, char *where, char *mail_cmd)
  *
  * Remove a message destination   
  */
-void rem_msg_dest(int dest_code, int msg_type, char *where)
+void rem_msg_dest(MSGS *msg, int dest_code, int msg_type, char *where)
 {
    DEST *d;
 
-   for (d=dest_chain; d; d=d->next) {
-      Dmsg2(200, "Remove_msg_dest d=%x where=%s\n", d, d->where);
+   for (d=msg->dest_chain; d; d=d->next) {
+      Dmsg2(200, "Remove_msg_dest d=%x where=%s\n", d, NPRT(d->where));
       if (bit_is_set(msg_type, d->msg_types) && (dest_code == d->dest_code) &&
          ((where == NULL && d->where == NULL) ||
                     (strcmp(where, d->where) == 0))) {  
@@ -248,201 +350,18 @@ void rem_msg_dest(int dest_code, int msg_type, char *where)
    }
 }
 
-/*
- * Concatenate a string (str) onto a message (msg)
- *  return new message pointer
- */
-static void add_str(char **base, char **msg, char *str)
-{
-   int len = strlen(str) + 1;
-   char *b, *m;
-
-   b = *base;
-   *base = (char *) check_pool_memory_size(*base, len);
-   m = *base - b + *msg;
-   while (*str) {
-      *m++ = *str++;
-   }
-   *msg = m;
-}
-
-/*
- * Convert Job Termination Status into a string
- */
-static char *job_status_to_str(int stat) 
-{
-   char *str;
-
-   switch (stat) {
-   case JS_Terminated:
-      str = "OK";
-      break;
-   case JS_Errored:
-      str = "Error";
-      break;
-   case JS_Cancelled:
-      str = "Cancelled";
-      break;
-   case JS_Differences:
-      str = "Differences";
-      break;
-   default:
-      str = "Unknown term code";
-      break;
-   }
-   return str;
-}
-
-
-/*
- * Convert Job Type into a string
- */
-static char *job_type_to_str(int type) 
-{
-   char *str;
-
-   switch (type) {
-   case JT_BACKUP:
-      str = "Backup";
-      break;
-   case JT_VERIFY:
-      str = "Verify";
-      break;
-   case JT_RESTORE:
-      str = "Restore";
-      break;
-   default:
-      str = "Unknown Job Type";
-      break;
-   }
-   return str;
-}
-
-/*
- * Convert Job Level into a string
- */
-static char *job_level_to_str(int level) 
-{
-   char *str;
-
-   switch (level) {
-   case L_FULL:
-      str = "full";
-      break;
-   case L_INCREMENTAL:
-      str = "incremental";
-      break;
-   case L_DIFFERENTIAL:
-      str = "differential";
-      break;
-   case L_LEVEL:
-      str = "level";
-      break;
-   case L_SINCE:
-      str = "since";
-      break;
-   case L_VERIFY_CATALOG:
-      str = "verify catalog";
-      break;
-   case L_VERIFY_INIT:
-      str = "verify init";
-      break;
-   case L_VERIFY_VOLUME:
-      str = "verify volume";
-      break;
-   case L_VERIFY_DATA:
-      str = "verify data";
-      break;
-   default:
-      str = "Unknown Job level";
-      break;
-   }
-   return str;
-}
-
-
-/*
- * Edit job codes into main command line
- *  %% = %
- *  %j = Job name
- *  %t = Job type (Backup, ...)
- *  %e = Job Exit code
- *  %l = job level
- *  %c = Client's name
- *  %r = Recipients
- *  %d = Director's name
- */
-static char *edit_job_codes(JCR *jcr, char *omsg, char *imsg, char *to)   
-{
-   char *p, *o, *str;
-   char add[3];
-
-   Dmsg1(200, "edit_job_codes: %s\n", imsg);
-   add[2] = 0;
-   o = omsg;
-   for (p=imsg; *p; p++) {
-      if (*p == '%') {
-        switch (*++p) {
-         case '%':
-            add[0] = '%';
-           add[1] = 0;
-           str = add;
-           break;
-         case 'j':                    /* Job name */
-           str = jcr->Job;
-           break;
-         case 'e':
-           str = job_status_to_str(jcr->JobStatus); 
-           break;
-         case 't':
-           str = job_type_to_str(jcr->JobType);
-           break;
-         case 'r':
-           str = to;
-           break;
-         case 'l':
-           str = job_level_to_str(jcr->level);
-           break;
-         case 'c':
-           str = jcr->client_name;
-           if (!str) {
-               str = "";
-           }
-           break;
-         case 'd':
-            str = my_name;            /* Director's name */
-           break;
-        default:
-            add[0] = '%';
-           add[1] = *p;
-           str = add;
-           break;
-        }
-      } else {
-        add[0] = *p;
-        add[1] = 0;
-        str = add;
-      }
-      Dmsg1(200, "add_str %s\n", str);
-      add_str(&omsg, &o, str);
-      *o = 0;
-      Dmsg1(200, "omsg=%s\n", omsg);
-   }
-   *o = 0;
-   return omsg;
-}
 
 /*
  * Create a unique filename for the mail command
  */
-static void make_unique_mail_filename(JCR *jcr, char **name, DEST *d)
+static void make_unique_mail_filename(JCR *jcr, POOLMEM *&name, DEST *d)
 {
    if (jcr) {
       Mmsg(name, "%s/%s.mail.%s.%d", working_directory, my_name,
-                jcr->Job, (int)d);
+                jcr->Job, (int)(long)d);
    } else {
       Mmsg(name, "%s/%s.mail.%s.%d", working_directory, my_name,
-                my_name, (int)d);
+                my_name, (int)(long)d);
    }
    Dmsg1(200, "mailname=%s\n", *name);
 }
@@ -450,48 +369,63 @@ static void make_unique_mail_filename(JCR *jcr, char **name, DEST *d)
 /*
  * Open a mail pipe
  */
-static FILE *open_mail_pipe(JCR *jcr, char **cmd, DEST *d)
+static BPIPE *open_mail_pipe(JCR *jcr, POOLMEM *&cmd, DEST *d)
 {
-   FILE *pfd;
-
-   if (d->mail_cmd && jcr) {
-      *cmd = edit_job_codes(jcr, *cmd, d->mail_cmd, d->where);
+   BPIPE *bpipe;
+   int use_bsmtp = (d->mail_cmd && jcr);
+       
+   if (use_bsmtp) {
+      cmd = edit_job_codes(jcr, cmd, d->mail_cmd, d->where);
    } else {
+#if 1
+      Mmsg(cmd, "/usr/lib/sendmail -F Bacula %s", d->where);
+#else
       Mmsg(cmd, "mail -s \"Bacula Message\" %s", d->where);
+#endif
    }
-   Dmsg1(200, "mailcmd=%s\n", cmd);
-   pfd = popen(*cmd, "w");
-   if (!pfd) {
-      Emsg2(M_ERROR, 0, "popen %s failed: ERR=%s\n", cmd, strerror(errno));
-      if (jcr) {
-         Jmsg(jcr, M_ERROR, 0, "mail popen %s failed: ERR=%s\n", cmd, strerror(errno));
-      }
-   } 
-   return pfd;
+   fflush(stdout);
+
+   if (!(bpipe = open_bpipe(cmd, 120, "rw"))) {
+      berrno be;
+      Jmsg(jcr, M_ERROR, 0, "open mail pipe %s failed: ERR=%s\n", 
+        cmd, be.strerror());
+   }
+
+#if 1
+   if (!use_bsmtp) {
+       fprintf(bpipe->wfd, "Subject: Bacula Message\r\n\r\n");
+   }
+#endif
+   
+   return bpipe;
 }
 
 /* 
- * Close the messages for this job, which means to close
+ * Close the messages for this Messages resource, which means to close
  *  any open files, and dispatch any pending email messages.
- *     
- * This closes messages only for this job, other jobs can   
- *   still send messages.
- * 
- * Note, we free our local message destination chain, but
- * the global chain remains allowing other jobs to
- * start.
  */
-void close_msg(void *vjcr)
+void close_msg(JCR *jcr)
 {
-   DEST *d, *old;
-   FILE *pfd;
-   char *cmd, *line;
-   int len;
-   JCR *jcr = (JCR *)vjcr;
+   MSGS *msgs;
+   DEST *d;
+   BPIPE *bpipe;
+   POOLMEM *cmd, *line;
+   int len, stat;
    
-   Dmsg0(200, "Close_msg\n");
-   cmd = (char *)get_pool_memory(PM_MESSAGE);
-   for (d=jcr->dest_chain; d; ) {
+   Dmsg1(050, "Close_msg jcr=0x%x\n", jcr);
+
+   if (jcr == NULL) {               /* NULL -> global chain */
+      msgs = daemon_msgs;
+   } else {
+      msgs = jcr->jcr_msgs;
+      jcr->jcr_msgs = NULL;
+   }
+   if (msgs == NULL) {
+      return;
+   }
+   Dmsg1(150, "===Begin close msg resource at 0x%x\n", msgs);
+   cmd = get_pool_memory(PM_MESSAGE);
+   for (d=msgs->dest_chain; d; ) {
       if (d->fd) {
         switch (d->dest_code) {
         case MD_FILE:
@@ -502,70 +436,113 @@ void close_msg(void *vjcr)
            break;
         case MD_MAIL:
         case MD_MAIL_ON_ERROR:
+            Dmsg0(150, "Got MD_MAIL or MD_MAIL_ON_ERROR\n");
            if (!d->fd) {
               break;
            }
-           if (d->dest_code == MD_MAIL_ON_ERROR && 
+           if (d->dest_code == MD_MAIL_ON_ERROR && jcr &&
                jcr->JobStatus == JS_Terminated) {
               goto rem_temp_file;
            }
            
-           pfd = open_mail_pipe(jcr, &cmd, d);
-           if (!pfd) {
+           if (!(bpipe=open_mail_pipe(jcr, cmd, d))) {
+               Pmsg0(000, "open mail pipe failed.\n");
               goto rem_temp_file;
            }
+            Dmsg0(150, "Opened mail pipe\n");
            len = d->max_len+10;
-           line = (char *)get_memory(len);
+           line = get_memory(len);
            rewind(d->fd);
-           while (fgets(line, len, d->fd)) {
-              fputs(line, pfd);
+           while (fgets(mp_chr(line), len, d->fd)) {
+              fputs(line, bpipe->wfd);
+           }
+           if (!close_wpipe(bpipe)) {       /* close write pipe sending mail */
+               Pmsg1(000, "close error: ERR=%s\n", strerror(errno));
+           }
+
+           /*
+             * Since we are closing all messages, before "recursing"
+            * make sure we are not closing the daemon messages, otherwise
+            * kaboom.
+            */
+           if (msgs != daemon_msgs) {
+              /* read what mail prog returned -- should be nothing */
+              while (fgets(mp_chr(line), len, bpipe->rfd)) {
+                  Jmsg1(jcr, M_INFO, 0, _("Mail prog: %s"), line);
+              }
+           }
+
+           stat = close_bpipe(bpipe);
+           if (stat != 0 && msgs != daemon_msgs) {
+              berrno be;
+              be.set_errno(stat);
+               Dmsg1(150, "Calling emsg. CMD=%s\n", cmd);
+               Jmsg2(jcr, M_ERROR, 0, _("Mail program terminated in error.\n"
+                                        "CMD=%s\n"
+                                        "ERR=%s\n"), cmd, be.strerror());
            }
-           pclose(pfd);            /* close pipe, sending mail */
            free_memory(line);
 rem_temp_file:
            /* Remove temp file */
            fclose(d->fd);
-           make_unique_mail_filename(jcr, &cmd, d);
-            Dmsg1(200, "unlink: %s\n", cmd);
-           unlink(cmd);
+           unlink(mp_chr(d->mail_filename));
+           free_pool_memory(d->mail_filename);
+           d->mail_filename = NULL;
+            Dmsg0(150, "end mail or mail on error\n");
            break;
         default:
            break;
         }
-        d->fd = 0;
+        d->fd = NULL;
+      }
+      d = d->next;                   /* point to next buffer */
+   }
+   free_pool_memory(cmd);
+   Dmsg0(150, "Done walking message chain.\n");
+   if (jcr) {
+      free_msgs_res(msgs);
+      msgs = NULL;
+   }
+   Dmsg0(150, "===End close msg resource\n");
+}
+
+/*
+ * Free memory associated with Messages resource  
+ */
+void free_msgs_res(MSGS *msgs)
+{
+   DEST *d, *old;
+
+   /* Walk down the message chain releasing allocated buffers */
+   for (d=msgs->dest_chain; d; ) {
+      if (d->where) {
+        free(d->where);
+      }
+      if (d->mail_cmd) {
+        free(d->mail_cmd);
       }
       old = d;                       /* save pointer to release */
       d = d->next;                   /* point to next buffer */
       free(old);                     /* free the destination item */
    }
-   free_pool_memory(cmd);
-   jcr->dest_chain = NULL;
+   msgs->dest_chain = NULL;
+   free(msgs);                       /* free the head */
 }
 
 
 /* 
  * Terminate the message handler for good. 
  * Release the global destination chain.
+ * 
+ * Also, clean up a few other items (cons, exepath). Note,
+ *   these really should be done elsewhere.
  */
 void term_msg()
 {
-   DEST *d, *n;
-   
-   for (d=dest_chain; d; d=n) {
-      if (d->fd) {
-        if (d->dest_code == MD_FILE || d->dest_code == MD_APPEND) {
-           fclose(d->fd);            /* close open file descriptor */
-        } else if (d->dest_code == MD_MAIL || d->dest_code == MD_MAIL_ON_ERROR) {
-           pclose(d->fd);            /* close open pipe */
-        }
-      }
-      n = d->next;
-      if (d->where)
-        free(d->where);              /* free destination address */
-      if (d->mail_cmd)
-        free(d->mail_cmd);
-      free(d);
-   }
+   Dmsg0(100, "Enter term_msg\n");
+   close_msg(NULL);                  /* close global chain */
+   free_msgs_res(daemon_msgs);       /* free the resources */
+   daemon_msgs = NULL;
    if (con_fd) {
       fflush(con_fd);
       fclose(con_fd);
@@ -579,6 +556,11 @@ void term_msg()
       free(exename);
       exename = NULL;
    }
+   if (trace_fd) {
+      fclose(trace_fd);
+      trace_fd = NULL;
+   }
+   term_last_jobs_list();
 }
 
 
@@ -586,129 +568,159 @@ void term_msg()
 /*
  * Handle sending the message to the appropriate place
  */
-void dispatch_message(void *vjcr, int type, int level, char *buf)
+void dispatch_message(JCR *jcr, int type, int level, char *msg)
 {
     DEST *d;   
-    char cmd[MAXSTRING], *mcmd;
-    JCR *jcr = (JCR *) vjcr;
+    char dt[MAX_TIME_LENGTH];
+    POOLMEM *mcmd;
     int len;
+    MSGS *msgs;
+    BPIPE *bpipe;
 
-    Dmsg2(200, "Enter dispatch_msg type=%d msg=%s\n", type, buf);
+    Dmsg2(800, "Enter dispatch_msg type=%d msg=%s\n", type, msg);
 
-    if (type == M_ABORT) {
-       fprintf(stdout, buf);         /* print this here to INSURE that it is printed */
+    if (type == M_ABORT || type == M_ERROR_TERM) {
+#ifndef HAVE_WIN32
+       fputs(msg, stdout);        /* print this here to INSURE that it is printed */
+       fflush(stdout);
+#endif
+#if !defined(HAVE_CONSOLE)
+#if defined(HAVE_CYGWIN) || defined(HAVE_WIN32)
+       /* If we don't exit on error, error messages are parsed by UA */
+       if (exit_on_error) {
+          MessageBox(NULL, msg, "Bacula", MB_OK);
+       }
+#endif
+#endif
     }
 
     /* Now figure out where to send the message */
+    msgs = NULL;
     if (jcr) {
-       d = jcr->dest_chain;          /* use job message chain */
-    } else {
-       d = dest_chain;               /* use global chain */
+       msgs = jcr->jcr_msgs;
+    } 
+    if (msgs == NULL) {
+       msgs = daemon_msgs;
     }
-    for ( ; d; d=d->next) {
+    for (d=msgs->dest_chain; d; d=d->next) {
        if (bit_is_set(type, d->msg_types)) {
          switch (d->dest_code) {
             case MD_CONSOLE:
-                Dmsg1(200, "CONSOLE for following err: %s\n", buf);
+                Dmsg1(800, "CONSOLE for following msg: %s", msg);
                if (!con_fd) {
                    con_fd = fopen(con_fname, "a+");
-                   Dmsg0(200, "Console file not open.\n");
+                   Dmsg0(800, "Console file not open.\n");
                }
                if (con_fd) {
-                  fcntl(fileno(con_fd), F_SETLKW);
+                  Pw(con_lock);      /* get write lock on console message file */
                   errno = 0;
-                  bstrftime(cmd, sizeof(cmd), time(NULL));
-                  len = strlen(cmd);
-                   cmd[len++] = ' ';
-                  fwrite(cmd, len, 1, con_fd);
-                  len = strlen(buf);
-                   if (len > 0 && buf[len-1] != '\n') {
-                      buf[len++] = '\n';
-                     buf[len] = 0;
+                  bstrftime(dt, sizeof(dt), time(NULL));
+                  len = strlen(dt);
+                   dt[len++] = ' ';
+                  fwrite(dt, len, 1, con_fd);
+                  len = strlen(msg);
+                  if (len > 0) {
+                     fwrite(msg, len, 1, con_fd);
+                      if (msg[len-1] != '\n') {
+                         fwrite("\n", 2, 1, con_fd);
+                     }
+                  } else {
+                      fwrite("\n", 2, 1, con_fd);
                   }
-                  fwrite(buf, len, 1, con_fd);
                   fflush(con_fd);
-                  fcntl(fileno(con_fd), F_UNLCK);
                   console_msg_pending = TRUE;
+                  Vw(con_lock);
                }
                break;
             case MD_SYSLOG:
-                Dmsg1(200, "SYSLOG for following err: %s\n", buf);
-               /* We really should do an openlog() here */
-               syslog(LOG_DAEMON|LOG_ERR, buf);
+                Dmsg1(800, "SYSLOG for collowing msg: %s\n", msg);
+               /*
+                * We really should do an openlog() here.  
+                */
+                syslog(LOG_DAEMON|LOG_ERR, "%s", msg);
                break;
             case MD_OPERATOR:
-                Dmsg1(200, "OPERATOR for following err: %s\n", buf);
-               mcmd = (char *) get_pool_memory(PM_MESSAGE);
-               d->fd = open_mail_pipe(jcr, &mcmd, d);
-               free_pool_memory(mcmd);
-               if (d->fd) {
-                  fputs(buf, d->fd);
+                Dmsg1(800, "OPERATOR for collowing msg: %s\n", msg);
+               mcmd = get_pool_memory(PM_MESSAGE);
+               if ((bpipe=open_mail_pipe(jcr, mcmd, d))) {
+                  int stat;
+                  fputs(msg, bpipe->wfd);
                   /* Messages to the operator go one at a time */
-                  pclose(d->fd);
+                  stat = close_bpipe(bpipe);
+                  if (stat != 0) {
+                     berrno be;
+                     be.set_errno(stat);
+                      Jmsg2(jcr, M_ERROR, 0, _("Operator mail program terminated in error.\n"
+                            "CMD=%s\n"
+                            "ERR=%s\n"), mcmd, be.strerror());
+                  }
                }
+               free_pool_memory(mcmd);
                break;
             case MD_MAIL:
             case MD_MAIL_ON_ERROR:
-                Dmsg1(200, "MAIL for following err: %s\n", buf);
+                Dmsg1(800, "MAIL for following msg: %s", msg);
                if (!d->fd) {
-                  char *name  = (char *) get_pool_memory(PM_MESSAGE);
-                  make_unique_mail_filename(jcr, &name, d);
-                   d->fd = fopen(name, "w+");
-                   Dmsg2(100, "Open mail file %d: %s\n", d->fd, name);
+                  POOLMEM *name = get_pool_memory(PM_MESSAGE);
+                  make_unique_mail_filename(jcr, name, d);
+                   d->fd = fopen(mp_chr(name), "w+");
                   if (!d->fd) {
-                      Emsg2(M_ERROR, 0, "fopen %s failed: ERR=%s\n", name, strerror(errno));
+                     d->fd = stdout;
+                      Jmsg2(jcr, M_ERROR, 0, "fopen %s failed: ERR=%s\n", name, strerror(errno));
+                     d->fd = NULL;
                      free_pool_memory(name);
                      break;
                   }
-                  free_pool_memory(name);
+                  d->mail_filename = name;
                }
-               len = strlen(buf);
+               len = strlen(msg);
                if (len > d->max_len) {
                   d->max_len = len;      /* keep max line length */
                }
-               fputs(buf, d->fd);
+               fputs(msg, d->fd);
                break;
             case MD_FILE:
-                Dmsg1(200, "FILE for following err: %s\n", buf);
+                Dmsg1(800, "FILE for following msg: %s", msg);
                if (!d->fd) {
                    d->fd = fopen(d->where, "w+");
                   if (!d->fd) {
-                      Emsg2(M_ERROR, 0, "fopen %s failed: ERR=%s\n", d->where, strerror(errno));
+                     d->fd = stdout;
+                      Jmsg2(jcr, M_ERROR, 0, "fopen %s failed: ERR=%s\n", d->where, strerror(errno));
+                     d->fd = NULL;
                      break;
                   }
                }
-               fputs(buf, d->fd);
+               fputs(msg, d->fd);
                break;
             case MD_APPEND:
-                Dmsg1(200, "APPEND for following err: %s\n", buf);
+                Dmsg1(800, "APPEND for following msg: %s", msg);
                if (!d->fd) {
                    d->fd = fopen(d->where, "a");
                   if (!d->fd) {
-                      Emsg2(M_ERROR, 0, "fopen %s failed: ERR=%s\n", d->where, strerror(errno));
+                     d->fd = stdout;
+                      Jmsg2(jcr, M_ERROR, 0, "fopen %s failed: ERR=%s\n", d->where, strerror(errno));
+                     d->fd = NULL;
                      break;
                   }
                }
-               fputs(buf, d->fd);
+               fputs(msg, d->fd);
                break;
             case MD_DIRECTOR:
-                Dmsg1(200, "DIRECTOR for following err: %s\n", buf);
+                Dmsg1(800, "DIRECTOR for following msg: %s", msg);
                if (jcr && jcr->dir_bsock && !jcr->dir_bsock->errors) {
-
-                  jcr->dir_bsock->msglen = Mmsg(&(jcr->dir_bsock->msg),
-                        "Jmsg Job=%s type=%d level=%d %s", jcr->Job,
-                        type, level, buf) + 1;
-                  bnet_send(jcr->dir_bsock);
+                   bnet_fsend(jcr->dir_bsock, "Jmsg Job=%s type=%d level=%d %s", 
+                     jcr->Job, type, level, msg);
                }
                break;
             case MD_STDOUT:
-                Dmsg1(200, "STDOUT for following err: %s\n", buf);
-               if (type != M_ABORT && type != M_FATAL)  /* already printed */
-                  fprintf(stdout, buf);
+                Dmsg1(800, "STDOUT for following msg: %s", msg);
+               if (type != M_ABORT && type != M_ERROR_TERM) { /* already printed */
+                  fputs(msg, stdout);
+               }
                break;
             case MD_STDERR:
-                Dmsg1(200, "STDERR for following err: %s\n", buf);
-               fprintf(stderr, buf);
+                Dmsg1(800, "STDERR for following msg: %s", msg);
+               fputs(msg, stderr);
                break;
             default:
                break;
@@ -720,7 +732,7 @@ void dispatch_message(void *vjcr, int type, int level, char *buf)
 
 /*********************************************************************
  *
- *  subroutine prints a debug message if the level number
+ *  This subroutine prints a debug message if the level number
  *  is less than or equal the debug_level. File and line numbers
  *  are included for more detail if desired, but not currently
  *  printed.
@@ -729,10 +741,10 @@ void dispatch_message(void *vjcr, int type, int level, char *buf)
  *  are not printed.
  */
 void 
-d_msg(char *file, int line, int level, char *fmt,...)
+d_msg(const char *file, int line, int level, const char *fmt,...)
 {
-    char      buf[MAXSTRING];
-    int       i;
+    char      buf[5000];
+    int       len;
     va_list   arg_ptr;
     int       details = TRUE;
 
@@ -741,80 +753,208 @@ d_msg(char *file, int line, int level, char *fmt,...)
        level = -level;
     }
 
-/*  printf("level=%d debug=%d fmt=%s\n", level, debug_level, fmt); */
-
     if (level <= debug_level) {
 #ifdef FULL_LOCATION
        if (details) {
-          sprintf(buf, "%s: %s:%d ", my_name, file, line);
-         i = strlen(buf);
+         /* visual studio passes the whole path to the file as well
+          * which makes for very long lines
+          */
+          const char *f = strrchr(file, '\\');
+         if (f) file = f + 1;
+          len = bsnprintf(buf, sizeof(buf), "%s: %s:%d ", my_name, file, line);
        } else {
-         i = 0;
+         len = 0;
        }
 #else
-       i = 0;
+       len = 0;
 #endif
        va_start(arg_ptr, fmt);
-       bvsnprintf(buf+i, sizeof(buf)-i, (char *)fmt, arg_ptr);
+       bvsnprintf(buf+len, sizeof(buf)-len, (char *)fmt, arg_ptr);
        va_end(arg_ptr);
 
-       fprintf(stdout, buf);
+       /* 
+        * Used the "trace on" command in the console to turn on
+        *  output to the trace file.  "trace off" will close the file.
+       */
+       if (trace) {
+         if (!trace_fd) {
+             bsnprintf(buf, sizeof(buf), "%s/bacula.trace", working_directory ? working_directory : ".");
+             trace_fd = fopen(buf, "a+");
+         }
+         if (trace_fd) {
+            fputs(buf, trace_fd);
+            fflush(trace_fd);
+         }
+       } else {   /* not tracing */
+         fputs(buf, stdout);
+       }
+    }
+}
+
+/*
+ * Set trace flag on/off. If argument is negative, there is no change 
+ */
+void set_trace(int trace_flag)
+{
+   if (trace_flag < 0) {
+      return;
+   } else if (trace_flag > 0) {
+      trace = true;
+   } else {
+      trace = false;
+   }
+   if (!trace && trace_fd) {
+      FILE *ltrace_fd = trace_fd;
+      trace_fd = NULL;
+      bmicrosleep(0, 100000);        /* yield to prevent seg faults */
+      fclose(ltrace_fd);
+   }
+}
+
+/*********************************************************************
+ *
+ *  This subroutine prints a message regardless of the debug level
+ *  
+ *  If the level is negative, the details of file and line number
+ *  are not printed.
+ */
+void 
+p_msg(const char *file, int line, int level, const char *fmt,...)
+{
+    char      buf[5000];
+    int       len;
+    va_list   arg_ptr;
+
+#ifdef FULL_LOCATION
+    if (level >= 0) {
+       len = bsnprintf(buf, sizeof(buf), "%s: %s:%d ", my_name, file, line);
+    } else {
+       len = 0;
+    }
+#else
+    len = 0;
+#endif
+    va_start(arg_ptr, fmt);
+    bvsnprintf(buf+len, sizeof(buf)-len, (char *)fmt, arg_ptr);
+    va_end(arg_ptr);
+    fputs(buf, stdout);
+}
+
+
+/*********************************************************************
+ *
+ *  subroutine writes a debug message to the trace file if the level number
+ *  is less than or equal the debug_level. File and line numbers
+ *  are included for more detail if desired, but not currently
+ *  printed.
+ *  
+ *  If the level is negative, the details of file and line number
+ *  are not printed.
+ */
+void 
+t_msg(const char *file, int line, int level, const char *fmt,...)
+{
+    char      buf[5000];
+    int       len;
+    va_list   arg_ptr;
+    int       details = TRUE;
+
+    if (level < 0) {
+       details = FALSE;
+       level = -level;
     }
+
+    if (level <= debug_level) {
+       if (!trace_fd) {
+          bsnprintf(buf, sizeof(buf), "%s/bacula.trace", working_directory);
+          trace_fd = fopen(buf, "a+");
+       }
+    
+#ifdef FULL_LOCATION
+       if (details) {
+          len = bsnprintf(buf, sizeof(buf), "%s: %s:%d ", my_name, file, line);
+       } else {
+         len = 0;
+       }
+#else
+       len = 0;
+#endif
+       va_start(arg_ptr, fmt);
+       bvsnprintf(buf+len, sizeof(buf)-len, (char *)fmt, arg_ptr);
+       va_end(arg_ptr);
+       if (trace_fd != NULL) {
+          fputs(buf, trace_fd);
+          fflush(trace_fd);
+       }
+   }
 }
 
 
+
 /* *********************************************************
  *
  * print an error message
  *
  */
 void 
-e_msg(char *file, int line, int type, int level, char *fmt,...)
+e_msg(const char *file, int line, int type, int level, const char *fmt,...)
 {
-    char     buf[1000];
+    char     buf[5000];
     va_list   arg_ptr;
-    int i;
+    int len;
 
     /* 
      * Check if we have a message destination defined. 
-     * We always report M_ABORT 
+     * We always report M_ABORT and M_ERROR_TERM 
      */
-    if (type != M_ABORT && !bit_is_set(type, send_msg))
+    if (!daemon_msgs || ((type != M_ABORT && type != M_ERROR_TERM) && 
+                        !bit_is_set(type, daemon_msgs->send_msg))) {
        return;                       /* no destination */
+    }
     switch (type) {
-       case M_ABORT:
-          sprintf(buf, "%s ABORTING due to ERROR in %s:%d\n", 
-                 my_name, file, line);
-         break;
-       case M_FATAL:
-         if (level == -1)            /* skip details */
-             sprintf(buf, "%s: Fatal Error because: ", my_name);
-         else
-             sprintf(buf, "%s: Fatal Error at %s:%d because:\n", my_name, file, line);
-         break;
-       case M_ERROR:
-         if (level == -1)            /* skip details */
-             sprintf(buf, "%s: Error: ", my_name);
-         else
-             sprintf(buf, "%s: Error in %s:%d ", my_name, file, line);
-         break;
-       case M_WARNING:
-          sprintf(buf, "%s: Warning: ", my_name);
-         break;
-       default:
-          sprintf(buf, "%s: ", my_name);
-         break;
+    case M_ABORT:
+       len = bsnprintf(buf, sizeof(buf), "%s: ABORTING due to ERROR in %s:%d\n", 
+              my_name, file, line);
+       break;
+    case M_ERROR_TERM:
+       len = bsnprintf(buf, sizeof(buf), "%s: ERROR TERMINATION at %s:%d\n", 
+              my_name, file, line);
+       break;
+    case M_FATAL:
+       if (level == -1)           /* skip details */
+          len = bsnprintf(buf, sizeof(buf), "%s: Fatal Error because: ", my_name);
+       else
+          len = bsnprintf(buf, sizeof(buf), "%s: Fatal Error at %s:%d because:\n", my_name, file, line);
+       break;
+    case M_ERROR:
+       if (level == -1)           /* skip details */
+          len = bsnprintf(buf, sizeof(buf), "%s: ERROR: ", my_name);
+       else
+          len = bsnprintf(buf, sizeof(buf), "%s: ERROR in %s:%d ", my_name, file, line);
+       break;
+    case M_WARNING:
+       len = bsnprintf(buf, sizeof(buf), "%s: Warning: ", my_name);
+       break;
+    case M_SECURITY:
+       len = bsnprintf(buf, sizeof(buf), "%s: Security violation: ", my_name);
+       break;
+    default:
+       len = bsnprintf(buf, sizeof(buf), "%s: ", my_name);
+       break;
     }
 
-    i = strlen(buf);
     va_start(arg_ptr, fmt);
-    bvsnprintf(buf+i, sizeof(buf)-i, (char *)fmt, arg_ptr);
+    bvsnprintf(buf+len, sizeof(buf)-len, (char *)fmt, arg_ptr);
     va_end(arg_ptr);
 
     dispatch_message(NULL, type, level, buf);
 
     if (type == M_ABORT) {
-       abort();
+       char *p = 0;
+       p[0] = 0;                     /* generate segmentation violation */
+    }
+    if ((type == M_ERROR_TERM) && exit_on_error) {
+       exit(1);
     }
 }
 
@@ -824,124 +964,314 @@ e_msg(char *file, int line, int type, int level, char *fmt,...)
  *
  */
 void 
-Jmsg(void *vjcr, int type, int level, char *fmt,...)
+Jmsg(JCR *jcr, int type, int level, const char *fmt,...)
 {
-    char     rbuf[2000];
-    char     *buf;
+    char     rbuf[5000];
     va_list   arg_ptr;
-    int i, len;
-    JCR *jcr = (JCR *) vjcr;
-    int typesave = type;
+    int len;
+    MSGS *msgs;
+    const char *job;
 
     
-    Dmsg1(200, "Enter Jmsg type=%d\n", type);
+    Dmsg1(800, "Enter Jmsg type=%d\n", type);
+
+    /* Special case for the console, which has a dir_bsock and JobId==0,
+     * in that case, we send the message directly back to the
+     * dir_bsock.  
+     */
+    if (jcr && jcr->JobId == 0 && jcr->dir_bsock) {
+       BSOCK *dir = jcr->dir_bsock;
+       va_start(arg_ptr, fmt);
+       dir->msglen = bvsnprintf(mp_chr(dir->msg), sizeof_pool_memory(dir->msg), 
+                               fmt, arg_ptr);
+       va_end(arg_ptr);
+       bnet_send(jcr->dir_bsock);
+       return;
+    }
+
+    msgs = NULL;
+    job = NULL;
+    if (jcr) {
+       msgs = jcr->jcr_msgs;
+       job = jcr->Job;
+    } 
+    if (!msgs) {
+       msgs = daemon_msgs;           /* if no jcr, we use daemon handler */
+    }
+    if (!job) {
+       job = "";                      /* Set null job name if none */
+    }
 
-    buf = rbuf;                   /* we are the Director */
     /* 
      * Check if we have a message destination defined. 
-     * We always report M_ABORT 
+     * We always report M_ABORT and M_ERROR_TERM 
      */
-    if (type != M_ABORT && !bit_is_set(type, jcr->send_msg)) {
-       Dmsg1(200, "No bit set for type %d\n", type);
+    if (msgs && (type != M_ABORT && type != M_ERROR_TERM) &&
+        !bit_is_set(type, msgs->send_msg)) {
        return;                       /* no destination */
     }
     switch (type) {
-       case M_ABORT:
-          sprintf(buf, "%s ABORTING due to ERROR\n", my_name);
-         break;
-       case M_FATAL:
-          sprintf(buf, "%s: Job %s Cancelled because: ", my_name, jcr->Job);
-         break;
-       case M_ERROR:
-          sprintf(buf, "%s: Job %s Error: ", my_name, jcr->Job);
-         break;
-       case M_WARNING:
-          sprintf(buf, "%s: Job %s Warning: ", my_name, jcr->Job);
-         break;
-       default:
-          sprintf(buf, "%s: ", my_name);
-         break;
+    case M_ABORT:
+       len = bsnprintf(rbuf, sizeof(rbuf), "%s ABORTING due to ERROR\n", my_name);
+       break;
+    case M_ERROR_TERM:
+       len = bsnprintf(rbuf, sizeof(rbuf), "%s ERROR TERMINATION\n", my_name);
+       break;
+    case M_FATAL:
+       len = bsnprintf(rbuf, sizeof(rbuf), "%s: %s Fatal error: ", my_name, job);
+       if (jcr) {
+         set_jcr_job_status(jcr, JS_FatalError);
+       }
+       break;
+    case M_ERROR:
+       len = bsnprintf(rbuf, sizeof(rbuf), "%s: %s Error: ", my_name, job);
+       if (jcr) {
+         jcr->Errors++;
+       }
+       break;
+    case M_WARNING:
+       len = bsnprintf(rbuf, sizeof(rbuf), "%s: %s Warning: ", my_name, job);
+       break;
+    case M_SECURITY:
+       len = bsnprintf(rbuf, sizeof(rbuf), "%s: %s Security violation: ", my_name, job);
+       break;
+    default:
+       len = bsnprintf(rbuf, sizeof(rbuf), "%s: ", my_name);
+       break;
     }
 
-    i = strlen(buf);
     va_start(arg_ptr, fmt);
-    len = bvsnprintf(buf+i, sizeof(rbuf)-i, fmt, arg_ptr);
+    bvsnprintf(rbuf+len,  sizeof(rbuf)-len, fmt, arg_ptr);
     va_end(arg_ptr);
 
-    ASSERT(typesave==type);          /* type trashed, compiler bug???? */
     dispatch_message(jcr, type, level, rbuf);
 
-    Dmsg3(500, "i=%d sizeof(rbuf)-i=%d len=%d\n", i, sizeof(rbuf)-i, len);
+    if (type == M_ABORT){
+       char *p = 0;
+       p[0] = 0;                     /* generate segmentation violation */
+    }
+    if ((type == M_ERROR_TERM) && exit_on_error) {
+       exit(1);
+    }
+}
 
-    if (type == M_ABORT)
-       abort();
+/*
+ * If we come here, prefix the message with the file:line-number,
+ *  then pass it on to the normal Jmsg routine.
+ */
+void j_msg(const char *file, int line, JCR *jcr, int type, int level, const char *fmt,...)
+{
+   va_list   arg_ptr;
+   int i, len, maxlen;
+   POOLMEM *pool_buf;
+
+   pool_buf = get_pool_memory(PM_EMSG);
+   i = Mmsg(pool_buf, "%s:%d ", file, line);
+
+   for (;;) {
+      maxlen = sizeof_pool_memory(pool_buf) - i - 1; 
+      va_start(arg_ptr, fmt);
+      len = bvsnprintf(pool_buf+i, maxlen, fmt, arg_ptr);
+      va_end(arg_ptr);
+      if (len < 0 || len >= (maxlen-5)) {
+        pool_buf = realloc_pool_memory(pool_buf, maxlen + i + maxlen/2);
+        continue;
+      }
+      break;
+   }
+
+   Jmsg(jcr, type, level, "%s", pool_buf);
+   free_memory(pool_buf);
 }
 
+
 /*
  * Edit a message into a Pool memory buffer, with file:lineno
- */
-int m_msg(char *file, int line, char **pool_buf, char *fmt, ...)
+ */                                                 
+int m_msg(const char *file, int line, POOLMEM **pool_buf, const char *fmt, ...)
+{
+   va_list   arg_ptr;
+   int i, len, maxlen;
+
+   i = sprintf(mp_chr(*pool_buf), "%s:%d ", file, line);
+
+   for (;;) {
+      maxlen = sizeof_pool_memory(*pool_buf) - i - 1; 
+      va_start(arg_ptr, fmt);
+      len = bvsnprintf(*pool_buf+i, maxlen, fmt, arg_ptr);
+      va_end(arg_ptr);
+      if (len < 0 || len >= (maxlen-5)) {
+        *pool_buf = realloc_pool_memory(*pool_buf, maxlen + i + maxlen/2);
+        continue;
+      }
+      break;
+   }
+   return len;
+}
+
+int m_msg(const char *file, int line, POOLMEM *&pool_buf, const char *fmt, ...)
 {
    va_list   arg_ptr;
    int i, len, maxlen;
 
-   sprintf(*pool_buf, "%s:%d ", file, line);
-   i = strlen(*pool_buf);
+   i = sprintf(pool_buf, "%s:%d ", file, line);
 
-again:
-   maxlen = sizeof_pool_memory(*pool_buf) - i - 1; 
-   va_start(arg_ptr, fmt);
-   len = bvsnprintf(*pool_buf+i, maxlen, fmt, arg_ptr);
-   va_end(arg_ptr);
-   if (len < 0 || len >= maxlen) {
-      *pool_buf = (char *) realloc_pool_memory(*pool_buf, maxlen + i + 200);
-      goto again;
+   for (;;) {
+      maxlen = sizeof_pool_memory(pool_buf) - i - 1; 
+      va_start(arg_ptr, fmt);
+      len = bvsnprintf(pool_buf+i, maxlen, fmt, arg_ptr);
+      va_end(arg_ptr);
+      if (len < 0 || len >= (maxlen-5)) {
+        pool_buf = realloc_pool_memory(pool_buf, maxlen + i + maxlen/2);
+        continue;
+      }
+      break;
    }
    return len;
 }
 
+
 /*
  * Edit a message into a Pool Memory buffer NO file:lineno
  *  Returns: string length of what was edited.
  */
-int Mmsg(char **pool_buf, char *fmt, ...)
+int Mmsg(POOLMEM **pool_buf, const char *fmt, ...)
 {
    va_list   arg_ptr;
    int len, maxlen;
 
-again:
-   maxlen = sizeof_pool_memory(*pool_buf) - 1; 
-   va_start(arg_ptr, fmt);
-   len = bvsnprintf(*pool_buf, maxlen, fmt, arg_ptr);
-   va_end(arg_ptr);
-   if (len < 0 || len >= maxlen) {
-      *pool_buf = (char *) realloc_pool_memory(*pool_buf, maxlen + 200);
-      goto again;
+   for (;;) {
+      maxlen = sizeof_pool_memory(*pool_buf) - 1; 
+      va_start(arg_ptr, fmt);
+      len = bvsnprintf(*pool_buf, maxlen, fmt, arg_ptr);
+      va_end(arg_ptr);
+      if (len < 0 || len >= (maxlen-5)) {
+        *pool_buf = realloc_pool_memory(*pool_buf, maxlen + maxlen/2);
+        continue;
+      }
+      break;
+   }
+   return len;
+}
+
+int Mmsg(POOLMEM *&pool_buf, const char *fmt, ...)
+{
+   va_list   arg_ptr;
+   int len, maxlen;
+
+   for (;;) {
+      maxlen = sizeof_pool_memory(pool_buf) - 1; 
+      va_start(arg_ptr, fmt);
+      len = bvsnprintf(pool_buf, maxlen, fmt, arg_ptr);
+      va_end(arg_ptr);
+      if (len < 0 || len >= (maxlen-5)) {
+        pool_buf = realloc_pool_memory(pool_buf, maxlen + maxlen/2);
+        continue;
+      }
+      break;
    }
    return len;
 }
 
 
-void j_msg(char *file, int line, void *jcr, int type, int level, char *fmt,...)
+static pthread_mutex_t msg_queue_mutex = PTHREAD_MUTEX_INITIALIZER;
+
+/*
+ * We queue messages rather than print them directly. This
+ *  is generally used in low level routines (msg handler, bnet)
+ *  to prevent recursion (i.e. if you are in the middle of 
+ *  sending a message, it is a bit messy to recursively call
+ *  yourself when the bnet packet is not reentrant).
+ */
+void Qmsg(JCR *jcr, int type, int level, const char *fmt,...)
 {
    va_list   arg_ptr;
-   int i, len, maxlen;
-   char *pool_buf;
+   int len, maxlen;
+   POOLMEM *pool_buf;
+   MQUEUE_ITEM *item;
+
+   pool_buf = get_pool_memory(PM_EMSG);
+
+   for (;;) {
+      maxlen = sizeof_pool_memory(pool_buf) - 1; 
+      va_start(arg_ptr, fmt);
+      len = bvsnprintf(pool_buf, maxlen, fmt, arg_ptr);
+      va_end(arg_ptr);
+      if (len < 0 || len >= (maxlen-5)) {
+        pool_buf = realloc_pool_memory(pool_buf, maxlen + maxlen/2);
+        continue;
+      }
+      break;
+   }
+   item = (MQUEUE_ITEM *)malloc(sizeof(MQUEUE_ITEM) + strlen(pool_buf) + 1);
+   item->type = type;
+   item->level = level;
+   strcpy(item->msg, pool_buf);  
+   P(msg_queue_mutex);
+   /* If no jcr or dequeuing send to daemon to avoid recursion */
+   if (!jcr || jcr->dequeuing) {
+      /* jcr==NULL => daemon message, safe to send now */
+      Jmsg(NULL, item->type, item->level, "%s", item->msg);
+      free(item);
+   } else {
+      /* Queue message for later sending */
+      jcr->msg_queue->append(item);
+//    Dmsg1(000, "queue item=%lu\n", (long unsigned)item);
+   }
+   V(msg_queue_mutex);
+   free_memory(pool_buf);
+}
+
+/*
+ * Dequeue messages 
+ */
+void dequeue_messages(JCR *jcr)
+{
+   MQUEUE_ITEM *item;
+   P(msg_queue_mutex);
+   jcr->dequeuing = true;
+   foreach_dlist(item, jcr->msg_queue) {
+//    Dmsg1(000, "dequeue item=%lu\n", (long unsigned)item);
+      Jmsg(jcr, item->type, item->level, "%s", item->msg);
+   }
+   jcr->msg_queue->destroy();
+   jcr->dequeuing = false;
+   V(msg_queue_mutex);
+}                                                
 
-   pool_buf = (char *) get_pool_memory(PM_EMSG);
-   sprintf(pool_buf, "%s:%d ", file, line);
-   i = strlen(pool_buf);
 
-again:
-   maxlen = sizeof_pool_memory(pool_buf) - i - 1; 
-   va_start(arg_ptr, fmt);
-   len = bvsnprintf(pool_buf+i, maxlen, fmt, arg_ptr);
-   va_end(arg_ptr);
-   if (len < 0 || len >= maxlen) {
-      pool_buf = (char *) realloc_pool_memory(pool_buf, maxlen + i + 200);
-      goto again;
+/*
+ * If we come here, prefix the message with the file:line-number,
+ *  then pass it on to the normal Qmsg routine.
+ */
+void q_msg(const char *file, int line, JCR *jcr, int type, int level, const char *fmt,...)
+{
+   va_list   arg_ptr;
+   int i, len, maxlen;
+   POOLMEM *pool_buf;
+
+   pool_buf = get_pool_memory(PM_EMSG);
+   i = Mmsg(pool_buf, "%s:%d ", file, line);
+
+   for (;;) {
+      maxlen = sizeof_pool_memory(pool_buf) - i - 1; 
+      va_start(arg_ptr, fmt);
+      len = bvsnprintf(pool_buf+i, maxlen, fmt, arg_ptr);
+      va_end(arg_ptr);
+      if (len < 0 || len >= (maxlen-5)) {
+        pool_buf = realloc_pool_memory(pool_buf, maxlen + i + maxlen/2);
+        continue;
+      }
+      break;
    }
 
-   Jmsg(jcr, type, level, pool_buf);
+   Qmsg(jcr, type, level, "%s", pool_buf);
    free_memory(pool_buf);
 }
+
+/* 
+ * Define if e_msg must exit when M_ERROR_TERM is received
+ */
+void set_exit_on_error(int value) {
+   exit_on_error = value;
+}