projects
/
tinc
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Move repeating MIN/MAX macros into dropin.h.
[tinc]
/
src
/
logger.c
diff --git
a/src/logger.c
b/src/logger.c
index
7fb1629
..
1a2e95f
100644
(file)
--- a/
src/logger.c
+++ b/
src/logger.c
@@
-28,8
+28,9
@@
#include "control_common.h"
#include "process.h"
#include "sptps.h"
#include "control_common.h"
#include "process.h"
#include "sptps.h"
+#include "compression.h"
-
in
t debug_level = DEBUG_NOTHING;
+
debug_
t debug_level = DEBUG_NOTHING;
static logmode_t logmode = LOGMODE_STDERR;
static pid_t logpid;
static FILE *logfile = NULL;
static logmode_t logmode = LOGMODE_STDERR;
static pid_t logpid;
static FILE *logfile = NULL;
@@
-37,22
+38,21
@@
static FILE *logfile = NULL;
static HANDLE loghandle = NULL;
#endif
static const char *logident = NULL;
static HANDLE loghandle = NULL;
#endif
static const char *logident = NULL;
-bool logcontrol = false;
+bool logcontrol = false;
// controlled by REQ_LOG <level>
int umbilical = 0;
int umbilical = 0;
-static void real_logger(int level, int priority, const char *message) {
+static bool should_log(debug_t level) {
+ return (level <= debug_level && logmode != LOGMODE_NULL) || logcontrol;
+}
+
+static void real_logger(debug_t level, int priority, const char *message) {
char timestr[32] = "";
static bool suppress = false;
char timestr[32] = "";
static bool suppress = false;
- // Bail out early if there is nothing to do.
if(suppress) {
return;
}
if(suppress) {
return;
}
- if(!logcontrol && (level > debug_level || logmode == LOGMODE_NULL)) {
- return;
- }
-
if(level <= debug_level) {
switch(logmode) {
case LOGMODE_STDERR:
if(level <= debug_level) {
switch(logmode) {
case LOGMODE_STDERR:
@@
-86,6
+86,7
@@
static void real_logger(int level, int priority, const char *message) {
break;
case LOGMODE_NULL:
break;
case LOGMODE_NULL:
+ default:
break;
}
break;
}
@@
-99,20
+100,20
@@
static void real_logger(int level, int priority, const char *message) {
suppress = true;
logcontrol = false;
suppress = true;
logcontrol = false;
- for list_each(connection_t, c, connection_list) {
+ for list_each(connection_t, c,
&
connection_list) {
if(!c->status.log) {
continue;
}
logcontrol = true;
if(!c->status.log) {
continue;
}
logcontrol = true;
- if(level > (c->outcompression >=
0
? c->outcompression : debug_level)) {
+ if(level > (c->outcompression >=
COMPRESS_NONE
? c->outcompression : debug_level)) {
continue;
}
continue;
}
-
in
t len = strlen(message);
+
size_
t len = strlen(message);
- if(send_request(c, "%d %d %
d
", CONTROL, REQ_LOG, len)) {
+ if(send_request(c, "%d %d %
zu
", CONTROL, REQ_LOG, len)) {
send_meta(c, message, len);
}
}
send_meta(c, message, len);
}
}
@@
-121,16
+122,20
@@
static void real_logger(int level, int priority, const char *message) {
}
}
}
}
-void logger(
in
t level, int priority, const char *format, ...) {
+void logger(
debug_
t level, int priority, const char *format, ...) {
va_list ap;
char message[1024] = "";
va_list ap;
char message[1024] = "";
+ if(!should_log(level)) {
+ return;
+ }
+
va_start(ap, format);
int len = vsnprintf(message, sizeof(message), format, ap);
message[sizeof(message) - 1] = 0;
va_end(ap);
va_start(ap, format);
int len = vsnprintf(message, sizeof(message), format, ap);
message[sizeof(message) - 1] = 0;
va_end(ap);
- if(len > 0 && len < sizeof(message) - 1 && message[len - 1] == '\n') {
+ if(len > 0 &&
(size_t)
len < sizeof(message) - 1 && message[len - 1] == '\n') {
message[len - 1] = 0;
}
message[len - 1] = 0;
}
@@
-138,13
+143,18
@@
void logger(int level, int priority, const char *format, ...) {
}
static void sptps_logger(sptps_t *s, int s_errno, const char *format, va_list ap) {
}
static void sptps_logger(sptps_t *s, int s_errno, const char *format, va_list ap) {
+ (void)s_errno;
char message[1024];
size_t msglen = sizeof(message);
char message[1024];
size_t msglen = sizeof(message);
+ if(!should_log(DEBUG_ALWAYS)) {
+ return;
+ }
+
int len = vsnprintf(message, msglen, format, ap);
message[sizeof(message) - 1] = 0;
int len = vsnprintf(message, msglen, format, ap);
message[sizeof(message) - 1] = 0;
- if(len > 0 && len < sizeof(message) - 1) {
+ if(len > 0 &&
(size_t)
len < sizeof(message) - 1) {
if(message[len - 1] == '\n') {
message[--len] = 0;
}
if(message[len - 1] == '\n') {
message[--len] = 0;
}
@@
-186,7
+196,7
@@
void openlogger(const char *ident, logmode_t mode) {
loghandle = RegisterEventSource(NULL, logident);
if(!loghandle) {
loghandle = RegisterEventSource(NULL, logident);
if(!loghandle) {
- fprintf(stderr, "Could not open log handle!");
+ fprintf(stderr, "Could not open log handle!
\n
");
logmode = LOGMODE_NULL;
}
logmode = LOGMODE_NULL;
}
@@
-199,6
+209,7
@@
void openlogger(const char *ident, logmode_t mode) {
#endif
case LOGMODE_NULL:
#endif
case LOGMODE_NULL:
+ default:
break;
}
break;
}
@@
-246,6
+257,7
@@
void closelogger(void) {
case LOGMODE_NULL:
case LOGMODE_STDERR:
case LOGMODE_NULL:
case LOGMODE_STDERR:
+ default:
break;
}
}
break;
}
}