Attachment 136764 Details for Bug 103737 – [8/11] bus: Try to get groups directly from credentials, not userdb

[patch] [8/11] bus: Try to get groups directly from credentials, not userdb

0008-bus-Try-to-get-groups-directly-from-credentials-not-.patch (text/plain), 1.65 KB, created by Simon McVittie on 2018-01-16 13:54:13 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Simon McVittie

Created: 2018-01-16 13:54:13 UTC

Size: 1.65 KB

patch

obsolete

>From faa3ee6f37efb0dbb3b387f73ca01cf8a9284632 Mon Sep 17 00:00:00 2001
>From: Simon McVittie <smcv@collabora.com>
>Date: Mon, 15 Jan 2018 19:45:39 +0000
>Subject: [PATCH 08/11] bus: Try to get groups directly from credentials, not
> userdb
>
>If we avoid consulting the userdb, then it's one less chance to
>deadlock.
>
>Signed-off-by: Simon McVittie <smcv@collabora.com>
>---
> bus/connection.c | 27 +++++++++++++++++++++++++++
> 1 file changed, 27 insertions(+)
>
>diff --git a/bus/connection.c b/bus/connection.c
>index 91b1966e..a3046dcb 100644
>--- a/bus/connection.c
>+++ b/bus/connection.c
>@@ -1032,11 +1032,38 @@ bus_connection_get_unix_groups  (DBusConnection   *connection,
>                                  int              *n_groups,
>                                  DBusError        *error)
> {
>+  const unsigned long *groups_borrowed = NULL;
>+  DBusCredentials *credentials;
>   unsigned long uid;
> 
>   *groups = NULL;
>   *n_groups = 0;
> 
>+  credentials = _dbus_connection_get_credentials (connection);
>+
>+  if (credentials != NULL &&
>+      _dbus_credentials_get_unix_gids (credentials, &groups_borrowed,
>+                                       n_groups))
>+    {
>+      int i;
>+
>+      /* We got the group IDs from SO_PEERGROUPS or equivalent - no
>+       * need to ask NSS */
>+
>+      *groups = dbus_new (dbus_gid_t, *n_groups);
>+
>+      if (groups == NULL)
>+        {
>+          BUS_SET_OOM (error);
>+          return FALSE;
>+        }
>+
>+      for (i = 0; i < *n_groups; i++)
>+        (*groups)[i] = groups_borrowed[i];
>+
>+      return TRUE;
>+    }
>+
>   if (dbus_connection_get_unix_user (connection, &uid))
>     {
>       if (!_dbus_unix_groups_from_uid (uid, groups, n_groups))
>-- 
>2.15.1
>

Actions: View | Splinter Review

Attachments on bug 103737: 136756 | 136757 | 136758 | 136759 | 136760 | 136762 | 136763 | 136764 | 136766 | 136767 | 136768 | 136770 | 137397 | 137398 | 137399 | 137400 | 138213 | 138221

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.