projects
/
tinc
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix UBSAN warnings about conversions and overflows.
[tinc]
/
src
/
logger.c
diff --git
a/src/logger.c
b/src/logger.c
index
9c02a3d
..
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;
@@
-40,11
+41,11
@@
static const char *logident = NULL;
bool logcontrol = false; // controlled by REQ_LOG <level>
int umbilical = 0;
bool logcontrol = false; // controlled by REQ_LOG <level>
int umbilical = 0;
-static bool should_log(
in
t level) {
+static bool should_log(
debug_
t level) {
return (level <= debug_level && logmode != LOGMODE_NULL) || logcontrol;
}
return (level <= debug_level && logmode != LOGMODE_NULL) || logcontrol;
}
-static void real_logger(
in
t level, int priority, const char *message) {
+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;
@@
-85,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;
}
@@
-98,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);
}
}
@@
-120,7
+122,7
@@
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] = "";
@@
-207,6
+209,7
@@
void openlogger(const char *ident, logmode_t mode) {
#endif
case LOGMODE_NULL:
#endif
case LOGMODE_NULL:
+ default:
break;
}
break;
}
@@
-254,6
+257,7
@@
void closelogger(void) {
case LOGMODE_NULL:
case LOGMODE_STDERR:
case LOGMODE_NULL:
case LOGMODE_STDERR:
+ default:
break;
}
}
break;
}
}