aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Wong <normalperson@yhbt.net>2008-08-19 03:01:39 -0700
committerEric Wong <normalperson@yhbt.net>2008-08-19 03:01:39 -0700
commit3befb84a6a7a95de7e0e94c2f0bd9936ecb60668 (patch)
tree6f466c0e9d71983beb2bff6a448782a37020a455
parent44d9f62f34e0561d83ea32941f0ea1b529b1490d (diff)
downloadmpd-3befb84a6a7a95de7e0e94c2f0bd9936ecb60668.tar.gz
mpd-3befb84a6a7a95de7e0e94c2f0bd9936ecb60668.tar.xz
mpd-3befb84a6a7a95de7e0e94c2f0bd9936ecb60668.zip
log.c: thread-safety for warning log
I'm really no fan of the warning log, it's too complex for how little it gets used; but fixing it is another problem.
-rw-r--r--src/log.c16
1 files changed, 12 insertions, 4 deletions
diff --git a/src/log.c b/src/log.c
index aac7349d1..2f4c1af0f 100644
--- a/src/log.c
+++ b/src/log.c
@@ -29,6 +29,7 @@ static unsigned int logLevel = LOG_LEVEL_LOW;
static int warningFlushed;
static int stdout_mode = 1;
static char *warningBuffer;
+static pthread_mutex_t warning_buffer_lock = PTHREAD_MUTEX_INITIALIZER;
static int out_fd = -1;
static int err_fd = -1;
static const char *out_filename;
@@ -81,7 +82,10 @@ static void do_log(FILE *fp, const char *fmt, va_list args)
void flushWarningLog(void)
{
- char *s = warningBuffer;
+ char *s;
+
+ pthread_mutex_lock(&warning_buffer_lock);
+ s = warningBuffer;
DEBUG("flushing warning messages\n");
@@ -98,8 +102,8 @@ void flushWarningLog(void)
warningBuffer = NULL;
}
-
warningFlushed = 1;
+ pthread_mutex_unlock(&warning_buffer_lock);
DEBUG("done flushing warning messages\n");
}
@@ -189,10 +193,14 @@ void WARNING(const char *fmt, ...)
{
va_list args;
va_start(args, fmt);
- if (warningFlushed) {
+
+ pthread_mutex_lock(&warning_buffer_lock);
+ if (warningFlushed)
do_log(stderr, fmt, args);
- } else
+ else
buffer_warning(fmt, args);
+ pthread_mutex_unlock(&warning_buffer_lock);
+
va_end(args);
}