From 9726b29e9811d12c0800a53f0a34dc26fe9b915c Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Fri, 21 May 2010 03:31:49 +0200 Subject: [PATCH] log: handle multi-line log messages properly --- fixme | 6 ++++-- src/log.c | 56 ++++++++++++++++++++++++++++++++++++------------------- 2 files changed, 41 insertions(+), 21 deletions(-) diff --git a/fixme b/fixme index 0c04ab8fe..22c5cc3a3 100644 --- a/fixme +++ b/fixme @@ -46,8 +46,9 @@ - uuidd DONE - nscd DONE + - dbus DONE - rpcbind (/var/run/rpcbind.sock!) - - dbus + - avahi-daemon (/var/run/avahi-daemon/socket) - rsyslog - cups - ssh CLASSIC @@ -55,7 +56,6 @@ - apache/samba - libvirtd (/var/run/libvirt/libvirt-sock-ro) - bluetoothd (/var/run/sdp! @/org/bluez/audio!) - - avahi-daemon (/var/run/avahi-daemon/socket) - distccd * teach dbus to talk to systemd when autospawning services @@ -63,6 +63,8 @@ * Figure out which signal handlers we actually have to reset in the forked off child +* loopback-setup is borked for ipv6 + Regularly: * look for close() vs. close_nointr() vs. close_nointr_nofail() diff --git a/src/log.c b/src/log.c index 7b4db4c77..1343b9d98 100644 --- a/src/log.c +++ b/src/log.c @@ -312,31 +312,49 @@ static int log_dispatch( const char*file, int line, const char *func, - const char *buffer) { + char *buffer) { - int r; + int r = 0; - if (log_target == LOG_TARGET_SYSLOG_OR_KMSG || - log_target == LOG_TARGET_SYSLOG) { + do { + char *e; + int k; - if ((r = write_to_syslog(level, file, line, func, buffer)) < 0) { - log_close_syslog(); - log_open_kmsg(); - } else if (r > 0) - return r; - } + buffer += strspn(buffer, NEWLINE); - if (log_target == LOG_TARGET_SYSLOG_OR_KMSG || - log_target == LOG_TARGET_KMSG) { + if (buffer[0] == 0) + break; - if ((r = write_to_kmsg(level, file, line, func, buffer)) < 0) { - log_close_kmsg(); - log_open_console(); - } else if (r > 0) - return r; - } + if ((e = strpbrk(buffer, NEWLINE))) + *(e++) = 0; - return write_to_console(level, file, line, func, buffer); + if (log_target == LOG_TARGET_SYSLOG_OR_KMSG || + log_target == LOG_TARGET_SYSLOG) { + + if ((r = write_to_syslog(level, file, line, func, buffer)) < 0) { + log_close_syslog(); + log_open_kmsg(); + } else if (r > 0) + r++; + } + + if (log_target == LOG_TARGET_SYSLOG_OR_KMSG || + log_target == LOG_TARGET_KMSG) { + + if ((r = write_to_kmsg(level, file, line, func, buffer)) < 0) { + log_close_kmsg(); + log_open_console(); + } else if (r > 0) + r++; + } + + if ((k = write_to_console(level, file, line, func, buffer)) < 0) + return k; + + buffer = e; + } while (buffer); + + return r; } int log_meta( -- 2.30.2