summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon McVittie <simon.mcvittie@collabora.co.uk>2011-03-15 15:42:56 +0000
committerSimon McVittie <simon.mcvittie@collabora.co.uk>2011-05-25 18:16:35 +0100
commitc9af1526d195f69e1ad794ea181f6b6341285a63 (patch)
tree47c027cc2a286451933851cd6ec33796d75e2d0e
parent7f0685a46d956f0471243de883f269064bf755f7 (diff)
downloaddbus-c9af1526d195f69e1ad794ea181f6b6341285a63.tar.gz
process_config_first_time_only: initialize syslog as intended.
On Linux, we previously called openlog() (with different options!) while initializing SELinux; leave SELinux messages as LOG_USER|LOG_INFO in case anyone was relying on that, but let the rest of our log messages come out as LOG_DAEMON. Also enable LOG_PERROR (copy syslog messages to stderr) unconditionally; we can make this an autoconf check if anyone's pet Unix doesn't have LOG_PERROR. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35358 Reviewed-by: Colin Walters <walters@verbum.org>
-rw-r--r--bus/bus.c2
-rw-r--r--bus/selinux.c3
-rw-r--r--dbus/dbus-sysdeps-util-unix.c2
3 files changed, 4 insertions, 3 deletions
diff --git a/bus/bus.c b/bus/bus.c
index 8b8ff92d..2c911c51 100644
--- a/bus/bus.c
+++ b/bus/bus.c
@@ -297,6 +297,8 @@ process_config_first_time_only (BusContext *context,
retval = FALSE;
auth_mechanisms = NULL;
+ _dbus_init_system_log ();
+
context->systemd_activation = systemd_activation;
/* Check for an existing pid file. Of course this is a race;
diff --git a/bus/selinux.c b/bus/selinux.c
index fe04063a..e94d8a8e 100644
--- a/bus/selinux.c
+++ b/bus/selinux.c
@@ -164,7 +164,7 @@ log_callback (const char *fmt, ...)
}
#endif /* HAVE_LIBAUDIT */
- vsyslog (LOG_INFO, fmt, ap);
+ vsyslog (LOG_USER | LOG_INFO, fmt, ap);
va_end(ap);
}
@@ -342,7 +342,6 @@ bus_selinux_full_init (void)
}
else
{
- openlog ("dbus", LOG_PERROR, LOG_USER);
_dbus_verbose ("Access Vector Cache (AVC) started.\n");
}
diff --git a/dbus/dbus-sysdeps-util-unix.c b/dbus/dbus-sysdeps-util-unix.c
index ba8ca7f8..a80f6430 100644
--- a/dbus/dbus-sysdeps-util-unix.c
+++ b/dbus/dbus-sysdeps-util-unix.c
@@ -425,7 +425,7 @@ _dbus_request_file_descriptor_limit (unsigned int limit)
void
_dbus_init_system_log (void)
{
- openlog ("dbus", LOG_PID, LOG_DAEMON);
+ openlog ("dbus", LOG_PID | LOG_PERROR, LOG_DAEMON);
}
/**
* Log a message to the system log file (e.g. syslog on Unix).