Don't modify the message string passed to logger, as it gets reused

diff --git a/src/libcharon/bus/listeners/file_logger.c b/src/libcharon/bus/listeners/file_logger.c
index 77af9e4ede516ba1d71bebaa66258c8c2d3780d2..9c8458eb5ce4560506ac8f0c9186a25087ebbf9d 100644 (file)
--- a/src/libcharon/bus/listeners/file_logger.c
+++ b/src/libcharon/bus/listeners/file_logger.c
@@ -62,10 +62,10 @@ struct private_file_logger_t {
 
 METHOD(logger_t, log_, void,
        private_file_logger_t *this, debug_t group, level_t level, int thread,
-       ike_sa_t* ike_sa, char *message)
+       ike_sa_t* ike_sa, const char *message)
 {
        char timestr[128], namestr[128] = "";
-       char *current = message, *next;
+       const char *current = message, *next;
        struct tm tm;
        time_t t;
 
@@ -95,24 +95,26 @@ METHOD(logger_t, log_, void,
 
        /* prepend a prefix in front of every line */
        this->mutex->lock(this->mutex);
-       while (current)
+       while (TRUE)
        {
                next = strchr(current, '\n');
-               if (next)
-               {
-                       *(next++) = '\0';
-               }
                if (this->time_format)
                {
-                       fprintf(this->out, "%s %.2d[%N]%s %s\n",
-                                       timestr, thread, debug_names, group, namestr, current);
+                       fprintf(this->out, "%s %.2d[%N]%s ",
+                                       timestr, thread, debug_names, group, namestr);
                }
                else
                {
-                       fprintf(this->out, "%.2d[%N]%s %s\n",
-                                       thread, debug_names, group, namestr, current);
+                       fprintf(this->out, "%.2d[%N]%s ",
+                                       thread, debug_names, group, namestr);
+               }
+               if (next == NULL)
+               {
+                       fprintf(this->out, "%s\n", current);
+                       break;
                }
-               current = next;
+               fprintf(this->out, "%.*s\n", (int)(next - current), current);
+               current = next + 1;
        }
        this->mutex->unlock(this->mutex);
 }

diff --git a/src/libcharon/bus/listeners/logger.h b/src/libcharon/bus/listeners/logger.h
index 030653635967828caf444c09523c1bc28069e3b0..3b99e7dc1b71d79ac6e00018598861ef5af47568 100644 (file)
--- a/src/libcharon/bus/listeners/logger.h
+++ b/src/libcharon/bus/listeners/logger.h
@@ -45,7 +45,7 @@ struct logger_t {
         * @param message       log message
         */
        void (*log)(logger_t *this, debug_t group, level_t level, int thread,
-                               ike_sa_t *ike_sa, char* message);
+                               ike_sa_t *ike_sa, const char *message);
 
        /**
         * Get the desired log level for a debug group.  This is called during

diff --git a/src/libcharon/bus/listeners/sys_logger.c b/src/libcharon/bus/listeners/sys_logger.c
index 9962fe0f6cae2f8ef3b83b567c007adad870997d..53fdefe890816a9396d09f724d308aef4a182a6b 100644 (file)
--- a/src/libcharon/bus/listeners/sys_logger.c
+++ b/src/libcharon/bus/listeners/sys_logger.c
@@ -57,10 +57,10 @@ struct private_sys_logger_t {
 
 METHOD(logger_t, log_, void,
        private_sys_logger_t *this, debug_t group, level_t level, int thread,
-       ike_sa_t* ike_sa, char *message)
+       ike_sa_t* ike_sa, const char *message)
 {
        char groupstr[4], namestr[128] = "";
-       char *current = message, *next;
+       const char *current = message, *next;
 
        /* cache group name and optional name string */
        snprintf(groupstr, sizeof(groupstr), "%N", debug_names, group);
@@ -81,16 +81,18 @@ METHOD(logger_t, log_, void,
 
        /* do a syslog for every line */
        this->mutex->lock(this->mutex);
-       while (current)
+       while (TRUE)
        {
                next = strchr(current, '\n');
-               if (next)
+               if (next == NULL)
                {
-                       *(next++) = '\0';
+                       syslog(this->facility | LOG_INFO, "%.2d[%s]%s %s\n",
+                                  thread, groupstr, namestr, current);
+                       break;
                }
-               syslog(this->facility|LOG_INFO, "%.2d[%s]%s %s\n",
-                          thread, groupstr, namestr, current);
-               current = next;
+               syslog(this->facility | LOG_INFO, "%.2d[%s]%s %.*s\n",
+                          thread, groupstr, namestr, (int)(next - current), current);
+               current = next + 1;
        }
        this->mutex->unlock(this->mutex);
 }

diff --git a/src/libcharon/control/controller.c b/src/libcharon/control/controller.c
index 0f98eb8c2fadfe3b439f0bba2cfd7b9e88b24f1e..77d73dba9b7f951f747d00f13642b952001b9240 100644 (file)
--- a/src/libcharon/control/controller.c
+++ b/src/libcharon/control/controller.c
@@ -209,7 +209,7 @@ static bool wait_for_listener(interface_job_t *job, u_int timeout)
 
 METHOD(logger_t, listener_log, void,
        interface_logger_t *this, debug_t group, level_t level, int thread,
-       ike_sa_t *ike_sa, char* message)
+       ike_sa_t *ike_sa, const char *message)
 {
        ike_sa_t *target;
 
@@ -627,7 +627,7 @@ METHOD(controller_t, terminate_child, status_t,
  * See header
  */
 bool controller_cb_empty(void *param, debug_t group, level_t level,
-                                                ike_sa_t *ike_sa, char *message)
+                                                ike_sa_t *ike_sa, const char *message)
 {
        return TRUE;
 }

diff --git a/src/libcharon/control/controller.h b/src/libcharon/control/controller.h
index 26614af32d37679ae91dbe59dd5550c465a2cdfb..222285cded2284f3a57714bbe5eebfcc41c4debf 100644 (file)
--- a/src/libcharon/control/controller.h
+++ b/src/libcharon/control/controller.h
@@ -34,7 +34,7 @@
  * @return                             FALSE to return from called controller method
  */
 typedef bool (*controller_cb_t)(void* param, debug_t group, level_t level,
-                                                               ike_sa_t* ike_sa, char* message);
+                                                               ike_sa_t* ike_sa, const char *message);
 
 /**
  * Empty callback function for controller_t methods.
@@ -43,7 +43,7 @@ typedef bool (*controller_cb_t)(void* param, debug_t group, level_t level,
  * this function to the controller methods.
  */
 bool controller_cb_empty(void *param, debug_t group, level_t level,
-                                                ike_sa_t *ike_sa, char *message);
+                                                ike_sa_t *ike_sa, const char *message);
 
 typedef struct controller_t controller_t;
 

diff --git a/src/libcharon/plugins/android/android_logger.c b/src/libcharon/plugins/android/android_logger.c
index 1f6042c562e12606d2b1051db95d25f3fe71a5b0..0c5f609f77f8aef8be12b519492bec7917333027 100644 (file)
--- a/src/libcharon/plugins/android/android_logger.c
+++ b/src/libcharon/plugins/android/android_logger.c
@@ -45,26 +45,27 @@ struct private_android_logger_t {
        mutex_t *mutex;
 };
 
-
 METHOD(logger_t, log_, void,
        private_android_logger_t *this, debug_t group, level_t level,
-       int thread, ike_sa_t* ike_sa, char *message)
+       int thread, ike_sa_t* ike_sa, const char *message)
 {
        int prio = level > 1 ? ANDROID_LOG_DEBUG : ANDROID_LOG_INFO;
        char sgroup[16];
-       char *current = message, *next;
+       const char *current = message, *next;
        snprintf(sgroup, sizeof(sgroup), "%N", debug_names, group);
        this->mutex->lock(this->mutex);
-       while (current)
+       while (TRUE)
        {       /* log each line separately */
                next = strchr(current, '\n');
-               if (next)
+               if (next == NULL)
                {
-                       *(next++) = '\0';
+                       __android_log_print(prio, "charon", "%.2d[%s] %s\n",
+                                                               thread, sgroup, current);
+                       break;
                }
-               __android_log_print(prio, "charon", "%.2d[%s] %s\n",
-                                                       thread, sgroup, current);
-               current = next;
+               __android_log_print(prio, "charon", "%.2d[%s] %.*s\n",
+                                                       thread, sgroup, (int)(next - current), current);
+               current = next + 1;
        }
        this->mutex->unlock(this->mutex);
 }

diff --git a/src/libcharon/plugins/sql/sql_logger.c b/src/libcharon/plugins/sql/sql_logger.c
index 093053516c1695f5a86f517799eedbf5729cf199..6db3258d23d1abddc514de92de3c29aa127cefdc 100644 (file)
--- a/src/libcharon/plugins/sql/sql_logger.c
+++ b/src/libcharon/plugins/sql/sql_logger.c
@@ -50,7 +50,7 @@ struct private_sql_logger_t {
 
 METHOD(logger_t, log_, void,
        private_sql_logger_t *this, debug_t group, level_t level, int thread,
-       ike_sa_t* ike_sa, char *message)
+       ike_sa_t* ike_sa, const char *message)
 {
        if (this->recursive->get(this->recursive))
        {