From 4d4cfeb9ae98c2d7b95b6f8984102661a9a08cf9 Mon Sep 17 00:00:00 2001
From: Frank Ueberschar <frank.ueberschar@bareos.com>
Date: Thu, 6 Feb 2020 11:56:16 +0100
Subject: [PATCH] messages: use const char* instead of char* for message string

---
 core/src/dird/dird.cc   | 4 +++-
 core/src/lib/message.cc | 9 ++++++---
 core/src/lib/message.h  | 9 ++++++---
 3 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/core/src/dird/dird.cc b/core/src/dird/dird.cc
index 4f3e4acefb0..54ac00fc779 100644
--- a/core/src/dird/dird.cc
+++ b/core/src/dird/dird.cc
@@ -157,7 +157,9 @@ static void ReloadJobEndCb(JobControlRecord* jcr, void* ctx)
  * access to the database. If the pointer is not defined (other daemons), then
  * writing the database is disabled.
  */
-static bool DirDbLogInsert(JobControlRecord* jcr, utime_t mtime, char* msg)
+static bool DirDbLogInsert(JobControlRecord* jcr,
+                           utime_t mtime,
+                           const char* msg)
 {
   int length;
   char ed1[50];
diff --git a/core/src/lib/message.cc b/core/src/lib/message.cc
index dab2d35f53e..c1b58715e14 100644
--- a/core/src/lib/message.cc
+++ b/core/src/lib/message.cc
@@ -72,7 +72,7 @@ job_code_callback_t message_job_code_callback = NULL;  // Only used by director
 static MessagesResource* daemon_msgs; /* Global messages */
 static char* catalog_db = NULL;       /* Database type */
 static const char* log_timestamp_format = "%d-%b %H:%M";
-static void (*message_callback)(int type, char* msg) = NULL;
+static void (*message_callback)(int type, const char* msg) = NULL;
 static FILE* trace_fd = NULL;
 #if defined(HAVE_WIN32)
 static bool trace = true;
@@ -137,7 +137,7 @@ static void DeliveryError(const char* fmt, ...)
   FreeMemory(pool_buf);
 }
 
-void RegisterMessageCallback(void msg_callback(int type, char* msg))
+void RegisterMessageCallback(void msg_callback(int type, const char* msg))
 {
   message_callback = msg_callback;
 }
@@ -621,7 +621,10 @@ static inline void SendToSyslog(int mode, const char* msg)
 /*
  * Handle sending the message to the appropriate place
  */
-void DispatchMessage(JobControlRecord* jcr, int type, utime_t mtime, char* msg)
+void DispatchMessage(JobControlRecord* jcr,
+                     int type,
+                     utime_t mtime,
+                     const char* msg)
 {
   char dt[MAX_TIME_LENGTH];
   POOLMEM* mcmd;
diff --git a/core/src/lib/message.h b/core/src/lib/message.h
index d5769c7b88e..64202d9549d 100644
--- a/core/src/lib/message.h
+++ b/core/src/lib/message.h
@@ -54,7 +54,7 @@ void SetLogTimestampFormat(const char* format);
 
 typedef bool (*db_log_insert_func)(JobControlRecord* jcr,
                                    utime_t mtime,
-                                   char* msg);
+                                   const char* msg);
 void SetDbLogInsertCallback(db_log_insert_func f);
 
 class MessagesResource;
@@ -79,7 +79,10 @@ void InitMsg(JobControlRecord* jcr,
 void TermMsg(void);
 void CloseMsg(JobControlRecord* jcr);
 void Jmsg(JobControlRecord* jcr, int type, utime_t mtime, const char* fmt, ...);
-void DispatchMessage(JobControlRecord* jcr, int type, utime_t mtime, char* buf);
+void DispatchMessage(JobControlRecord* jcr,
+                     int type,
+                     utime_t mtime,
+                     const char* buf);
 void InitConsoleMsg(const char* wd);
 void DequeueMessages(JobControlRecord* jcr);
 void SetTrace(int trace_flag);
@@ -89,6 +92,6 @@ bool GetHangup(void);
 void SetTimestamp(int timestamp_flag);
 bool GetTimestamp(void);
 void SetDbType(const char* name);
-void RegisterMessageCallback(void msg_callback(int type, char* msg));
+void RegisterMessageCallback(void msg_callback(int type, const char* msg));
 
 #endif /* BAREOS_LIB_MESSAGE_H_ */