_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

diff --git a/bus/audit.c b/bus/audit.c
index 7705e4250f49836113bced69020383799fa0f1c3..91f2c5cf65eff82814595a6c5dbb786d9f996ff5 100644 (file)
--- 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),