diff options
author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2015-01-26 20:06:48 +0000 |
---|---|---|
committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2015-02-05 14:54:31 +0000 |
commit | 03c5e161752fe1ff4925955800ca9c78d09a6e0c (patch) | |
tree | 3e4a887723dbfdf960613a67e9918ba4a065cafb /bus/driver.c | |
parent | aaea59916398d1c590490edb0471a01bcf20e6d7 (diff) | |
download | dbus-03c5e161752fe1ff4925955800ca9c78d09a6e0c.tar.gz |
CVE-2015-0245: discard forged ActivationFailure messages
Without this code change, non-systemd processes can make dbus-daemon
think systemd failed to activate a system service, resulting in an
error reply back to the requester. In practice we can address this in
system.conf by only allowing root to forge these messages, but this
check is the real solution, particularly on systems where root is
not all-powerful.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=88811
Reviewed-by: Alban Crequy
Reviewed-by: David King
Reviewed-by: Philip Withnall
Diffstat (limited to 'bus/driver.c')
-rw-r--r-- | bus/driver.c | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/bus/driver.c b/bus/driver.c index 9551f3d4..f494170c 100644 --- a/bus/driver.c +++ b/bus/driver.c @@ -2061,8 +2061,26 @@ bus_driver_handle_message (DBusConnection *connection, if (dbus_message_is_signal (message, "org.freedesktop.systemd1.Activator", "ActivationFailure")) { BusContext *context; + DBusConnection *systemd; context = bus_connection_get_context (connection); + systemd = bus_driver_get_owner_of_name (connection, + "org.freedesktop.systemd1"); + + if (systemd != connection) + { + const char *attacker; + + attacker = bus_connection_get_name (connection); + bus_context_log (context, DBUS_SYSTEM_LOG_SECURITY, + "Ignoring forged ActivationFailure message from " + "connection %s (%s)", + attacker ? attacker : "(unauthenticated)", + bus_connection_get_loginfo (connection)); + /* ignore it */ + return TRUE; + } + return dbus_activation_systemd_failure(bus_context_get_activation(context), message); } |