Attachment 86078 Details for Bug 69538 – 0003-PolkitSystemBusName-Add-public-API-to-retrieve-Unix-.patch

[patch] 0003-PolkitSystemBusName-Add-public-API-to-retrieve-Unix-.patch

0003-PolkitSystemBusName-Add-public-API-to-retrieve-Unix-.patch (text/plain), 6.01 KB, created by Miloslav Trmac on 2013-09-18 17:24:00 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Miloslav Trmac

Created: 2013-09-18 17:24:00 UTC

Size: 6.01 KB

patch

obsolete

>From ce966c68e62c7772a378001be5e96bd4abbf3c42 Mon Sep 17 00:00:00 2001
>From: Colin Walters <walters@verbum.org>
>Date: Wed, 21 Aug 2013 12:23:55 -0400
>Subject: [PATCH 3/4] PolkitSystemBusName: Add public API to retrieve Unix user
>
>And change the duplicated code in the backend session monitors to use
>it.
>---
> src/polkit/polkitsystembusname.c                   |   55 ++++++++++++++++++++
> .../polkitbackendsessionmonitor-systemd.c          |   20 +-------
> src/polkitbackend/polkitbackendsessionmonitor.c    |   20 +-------
> 3 files changed, 57 insertions(+), 38 deletions(-)
>
>diff --git a/src/polkit/polkitsystembusname.c b/src/polkit/polkitsystembusname.c
>index 2a297c4..141e72a 100644
>--- a/src/polkit/polkitsystembusname.c
>+++ b/src/polkit/polkitsystembusname.c
>@@ -396,3 +396,58 @@ polkit_system_bus_name_get_process_sync (PolkitSystemBusName  *system_bus_name,
>   return ret;
> }
> 
>+/**
>+ * polkit_system_bus_name_get_user_sync:
>+ * @system_bus_name: A #PolkitSystemBusName.
>+ * @cancellable: (allow-none): A #GCancellable or %NULL.
>+ * @error: (allow-none): Return location for error or %NULL.
>+ *
>+ * Synchronously gets a #PolkitUnixUser object for @system_bus_name;
>+ * the calling thread is blocked until a reply is received.
>+ *
>+ * Returns: (allow-none) (transfer full): A #PolkitUnixUser object or %NULL if @error is set.
>+ **/
>+PolkitSubject *
>+polkit_system_bus_name_get_user_sync (PolkitSystemBusName  *system_bus_name,
>+				      GCancellable         *cancellable,
>+				      GError              **error)
>+{
>+  GDBusConnection *connection;
>+  PolkitUnixUser *ret;
>+  GVariant *result;
>+  guint32 uid;
>+
>+  g_return_val_if_fail (POLKIT_IS_SYSTEM_BUS_NAME (system_bus_name), NULL);
>+  g_return_val_if_fail (cancellable == NULL || G_IS_CANCELLABLE (cancellable), NULL);
>+  g_return_val_if_fail (error == NULL || *error == NULL, NULL);
>+
>+  ret = NULL;
>+
>+  connection = g_bus_get_sync (G_BUS_TYPE_SYSTEM, cancellable, error);
>+  if (connection == NULL)
>+    goto out;
>+
>+  result = g_dbus_connection_call_sync (connection,
>+                                        "org.freedesktop.DBus",       /* name */
>+                                        "/org/freedesktop/DBus",      /* object path */
>+                                        "org.freedesktop.DBus",       /* interface name */
>+                                        "GetConnectionUnixUser",      /* method */
>+                                        g_variant_new ("(s)", system_bus_name->name),
>+                                        G_VARIANT_TYPE ("(u)"),
>+                                        G_DBUS_CALL_FLAGS_NONE,
>+                                        -1,
>+                                        cancellable,
>+                                        error);
>+  if (result == NULL)
>+    goto out;
>+
>+  g_variant_get (result, "(u)", &uid);
>+  g_variant_unref (result);
>+
>+  ret = polkit_unix_user_new (uid);
>+
>+ out:
>+  if (connection != NULL)
>+    g_object_unref (connection);
>+  return ret;
>+}
>diff --git a/src/polkitbackend/polkitbackendsessionmonitor-systemd.c b/src/polkitbackend/polkitbackendsessionmonitor-systemd.c
>index 58593c3..8d8479e 100644
>--- a/src/polkitbackend/polkitbackendsessionmonitor-systemd.c
>+++ b/src/polkitbackend/polkitbackendsessionmonitor-systemd.c
>@@ -277,25 +277,7 @@ polkit_backend_session_monitor_get_user_for_subject (PolkitBackendSessionMonitor
>     }
>   else if (POLKIT_IS_SYSTEM_BUS_NAME (subject))
>     {
>-      GVariant *result;
>-
>-      result = g_dbus_connection_call_sync (monitor->system_bus,
>-                                            "org.freedesktop.DBus",
>-                                            "/org/freedesktop/DBus",
>-                                            "org.freedesktop.DBus",
>-                                            "GetConnectionUnixUser",
>-                                            g_variant_new ("(s)", polkit_system_bus_name_get_name (POLKIT_SYSTEM_BUS_NAME (subject))),
>-                                            G_VARIANT_TYPE ("(u)"),
>-                                            G_DBUS_CALL_FLAGS_NONE,
>-                                            -1, /* timeout_msec */
>-                                            NULL, /* GCancellable */
>-                                            error);
>-      if (result == NULL)
>-        goto out;
>-      g_variant_get (result, "(u)", &uid);
>-      g_variant_unref (result);
>-
>-      ret = polkit_unix_user_new (uid);
>+      ret = polkit_system_bus_name_get_user_sync (POLKIT_SYSTEM_BUS_NAME (subject));
>     }
>   else if (POLKIT_IS_UNIX_SESSION (subject))
>     {
>diff --git a/src/polkitbackend/polkitbackendsessionmonitor.c b/src/polkitbackend/polkitbackendsessionmonitor.c
>index 9c331b6..73f2689 100644
>--- a/src/polkitbackend/polkitbackendsessionmonitor.c
>+++ b/src/polkitbackend/polkitbackendsessionmonitor.c
>@@ -306,25 +306,7 @@ polkit_backend_session_monitor_get_user_for_subject (PolkitBackendSessionMonitor
>     }
>   else if (POLKIT_IS_SYSTEM_BUS_NAME (subject))
>     {
>-      GVariant *result;
>-
>-      result = g_dbus_connection_call_sync (monitor->system_bus,
>-                                            "org.freedesktop.DBus",
>-                                            "/org/freedesktop/DBus",
>-                                            "org.freedesktop.DBus",
>-                                            "GetConnectionUnixUser",
>-                                            g_variant_new ("(s)", polkit_system_bus_name_get_name (POLKIT_SYSTEM_BUS_NAME (subject))),
>-                                            G_VARIANT_TYPE ("(u)"),
>-                                            G_DBUS_CALL_FLAGS_NONE,
>-                                            -1, /* timeout_msec */
>-                                            NULL, /* GCancellable */
>-                                            error);
>-      if (result == NULL)
>-        goto out;
>-      g_variant_get (result, "(u)", &uid);
>-      g_variant_unref (result);
>-
>-      ret = polkit_unix_user_new (uid);
>+      ret = polkit_system_bus_name_get_user_sync (POLKIT_SYSTEM_BUS_NAME (subject));
>     }
>   else if (POLKIT_IS_UNIX_SESSION (subject))
>     {
>-- 
>1.7.1
>

Actions: View | Splinter Review

Attachments on bug 69538: 86078 | 86079 | 88855 | 88856 | 88857 | 88858 | 88934 | 88944 | 88945 | 89028

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.