diff options
| author | Simon McVittie <smcv@debian.org> | 2016-07-21 10:55:06 +0100 |
|---|---|---|
| committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2016-08-12 17:34:29 +0100 |
| commit | b7d280f623b716eb437f9742d5ff0b7229dfeacf (patch) | |
| tree | 8d8ff6e12381f5806189eaf5e7782f0eb5a393b2 | |
| parent | 76ae52735ee9b574976f43426303636c41203d4c (diff) | |
_dbus_change_to_daemon_user (audit code path): set DBusError correctly
The other error cases here all set the error instead of just warning.
Signed-off-by: Simon McVittie <smcv@debian.org>
Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=97009
| -rw-r--r-- | bus/audit.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/bus/audit.c b/bus/audit.c index 7705e4250..91f2c5cf6 100644 --- a/bus/audit.c +++ b/bus/audit.c @@ -166,8 +166,9 @@ _dbus_change_to_daemon_user (const char *user, _dbus_strerror (errno)); break; case -5: - _dbus_warn ("Failed to drop supplementary groups: %s\n", - _dbus_strerror (errno)); + dbus_set_error (error, _dbus_error_from_errno (errno), + "Failed to drop supplementary groups: %s", + _dbus_strerror (errno)); break; case -6: dbus_set_error (error, _dbus_error_from_errno (errno), |