--- src/screen.c Fri Jan 25 18:46:56 2008 +0200
+++ src/screen.c Sat Jan 26 01:28:53 2008 +0200
@@ -228,10 +228,29 @@ static void
flag_vputf(DCDisplayFlag flag, const char *format, va_list args)
+ if (log_fh != NULL && log_flags & flag) {
+ time_t now = time(NULL);
- //va_copy(args2, args);
+ if (NULL != localtime_r(&now, &_tm) && 0 != strftime(c_time, 1023, "%d.%m.%Y %H:%M:%S", &_tm)) {
+ fprintf(log_fh, "%s ", c_time);
+ // args would become undefined after xvasprintf() call, so make a copy
+ // of args to use it later below.
+ va_copy(args_dup, args);
+ char* msg = xvasprintf(format, args_dup);
+ char* log_msg = main_to_log_string(msg);
+ fprintf(log_fh, log_msg);
if (display_flags & flag) {
if (screen_state == SCREEN_SUSPENDED) {
ptrv_append(suspend_msgs, xvasprintf(format, args));
@@ -243,21 +262,6 @@ flag_vputf(DCDisplayFlag flag, const cha
- if (log_fh != NULL && log_flags & flag) {
- time_t now = time(NULL);
- if (NULL != localtime_r(&now, &_tm) && 0 != strftime(c_time, 1023, "%d.%m.%Y %H:%M:%S", &_tm)) {
- fprintf(log_fh, "%s ", c_time);
- char* msg = xvasprintf(format, args);
- char* log_msg = main_to_log_string(msg);
- fprintf(log_fh, log_msg);