Avoid allocating packet buffers unnecessarily.

[meshlink] / src / logger.c

diff --git a/src/logger.c b/src/logger.c
index bac5afac9f3faa363441108c1af5c5fb6cfe0128..78d629ee2e5305a57e812522e7bb79c04f5ed1d2 100644 (file)
--- a/src/logger.c
+++ b/src/logger.c
@@ -1,7 +1,6 @@
 /*
     logger.c -- logging code
-    Copyright (C) 2004-2013 Guus Sliepen <guus@meshlink.io>
-                  2004-2005 Ivo Timmermans
+    Copyright (C) 2014-2017 Guus Sliepen <guus@meshlink.io>
 
     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
@@ -20,125 +19,38 @@
 
 #include "system.h"
 
-#include "conf.h"
-#include "meta.h"
 #include "logger.h"
-#include "connection.h"
+#include "meshlink_internal.h"
 #include "sptps.h"
 
-debug_t debug_level = DEBUG_NOTHING;
-static logmode_t logmode = LOGMODE_STDERR;
-static pid_t logpid;
-#ifdef HAVE_MINGW
-static HANDLE loghandle = NULL;
-#endif
-static const char *logident = NULL;
+// TODO: refactor logging code to use a meshlink_handle_t *.
+void logger(meshlink_handle_t *mesh, meshlink_log_level_t level, const char *format, ...) {
+       assert(format);
 
-static void real_logger(int level, int priority, const char *message) {
-       char timestr[32] = "";
-       static bool suppress = false;
-
-       // Bail out early if there is nothing to do.
-       if(suppress)
-               return;
-
-       if(level > debug_level || logmode == LOGMODE_NULL)
-               return;
-
-       if(level <= debug_level) {
-               switch(logmode) {
-                       case LOGMODE_STDERR:
-                               fprintf(stderr, "%s\n", message);
-                               fflush(stderr);
-                               break;
-                       case LOGMODE_SYSLOG:
-#ifdef HAVE_MINGW
-                               {
-                                       const char *messages[] = {message};
-                                       ReportEvent(loghandle, priority, 0, 0, NULL, 1, 0, messages, NULL);
-                               }
-#else
-#ifdef HAVE_SYSLOG_H
-                               syslog(priority, "%s", message);
-#endif
-#endif
-                               break;
-                       case LOGMODE_NULL:
-                               break;
+       if(mesh) {
+               if(level < mesh->log_level || !mesh->log_cb) {
+                       return;
+               }
+       } else {
+               if(level < global_log_level || !global_log_cb) {
+                       return;
                }
        }
-}
 
-void logger(int level, int priority, const char *format, ...) {
        va_list ap;
        char message[1024] = "";
 
        va_start(ap, format);
-       int len = vsnprintf(message, sizeof message, format, ap);
+       int len = vsnprintf(message, sizeof(message), format, ap);
        va_end(ap);
 
-       if(len > 0 && len < sizeof message && message[len - 1] == '\n')
-               message[len - 1] = 0;
-
-       real_logger(level, priority, message);
-}
-
-static void sptps_logger(sptps_t *s, int s_errno, const char *format, va_list ap) {
-       char message[1024] = "";
-       int len = vsnprintf(message, sizeof message, format, ap);
-       if(len > 0 && len < sizeof message && message[len - 1] == '\n')
+       if(len > 0 && (size_t)len < sizeof(message) && message[len - 1] == '\n') {
                message[len - 1] = 0;
-
-       real_logger(DEBUG_ALWAYS, LOG_ERR, message);
-}
-
-void openlogger(const char *ident, logmode_t mode) {
-       logident = ident;
-       logmode = mode;
-
-       switch(mode) {
-               case LOGMODE_STDERR:
-                       logpid = getpid();
-                       break;
-               case LOGMODE_SYSLOG:
-#ifdef HAVE_MINGW
-                       loghandle = RegisterEventSource(NULL, logident);
-                       if(!loghandle) {
-                               fprintf(stderr, "Could not open log handle!");
-                               logmode = LOGMODE_NULL;
-                       }
-                       break;
-#else
-#ifdef HAVE_SYSLOG_H
-                       openlog(logident, LOG_CONS | LOG_PID, LOG_DAEMON);
-                       break;
-#endif
-#endif
-               case LOGMODE_NULL:
-                       break;
        }
 
-       if(logmode != LOGMODE_NULL)
-               sptps_log = sptps_logger;
-       else
-               sptps_log = sptps_log_quiet;
-}
-
-void closelogger(void) {
-       switch(logmode) {
-               case LOGMODE_SYSLOG:
-#ifdef HAVE_MINGW
-                       DeregisterEventSource(loghandle);
-                       break;
-#else
-#ifdef HAVE_SYSLOG_H
-                       closelog();
-                       break;
-#endif
-#endif
-               case LOGMODE_NULL:
-               case LOGMODE_STDERR:
-                       break;
-                       break;
+       if(mesh) {
+               mesh->log_cb(mesh, level, message);
+       } else {
+               global_log_cb(NULL, level, message);
        }
 }