Attachment 89028 Details for Bug 69538 – 0001-Port-internals-non-deprecated-PolkitProcess-API-wher.patch

[patch] 0001-Port-internals-non-deprecated-PolkitProcess-API-wher.patch

0001-Port-internals-non-deprecated-PolkitProcess-API-wher.patch (text/plain), 2.34 KB, created by Colin Walters on 2013-11-11 14:09:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Colin Walters

Created: 2013-11-11 14:09:32 UTC

Size: 2.34 KB

patch

obsolete

>From 4773fccdb57c5a72e22cd57f5f0f9305fa44d952 Mon Sep 17 00:00:00 2001
>From: Colin Walters <walters@verbum.org>
>Date: Sat, 9 Nov 2013 13:48:21 -0500
>Subject: [PATCH] Port internals non-deprecated PolkitProcess API where possible
>
>We can't port everything, but in PolkitPermission and these test
>cases, we can use _for_owner() with the right information.
>---
> src/polkit/polkitpermission.c                      |    2 +-
> test/polkitbackend/test-polkitbackendjsauthority.c |    8 ++++----
> 2 files changed, 5 insertions(+), 5 deletions(-)
>
>diff --git a/src/polkit/polkitpermission.c b/src/polkit/polkitpermission.c
>index 22d195f..f8a666e 100644
>--- a/src/polkit/polkitpermission.c
>+++ b/src/polkit/polkitpermission.c
>@@ -122,7 +122,7 @@ polkit_permission_constructed (GObject *object)
>   PolkitPermission *permission = POLKIT_PERMISSION (object);
> 
>   if (permission->subject == NULL)
>-    permission->subject = polkit_unix_process_new (getpid ());
>+    permission->subject = polkit_unix_process_new_for_owner (getpid (), 0, getuid ());
> 
>   if (G_OBJECT_CLASS (polkit_permission_parent_class)->constructed != NULL)
>     G_OBJECT_CLASS (polkit_permission_parent_class)->constructed (object);
>diff --git a/test/polkitbackend/test-polkitbackendjsauthority.c b/test/polkitbackend/test-polkitbackendjsauthority.c
>index a4de6b1..dfb894f 100644
>--- a/test/polkitbackend/test-polkitbackendjsauthority.c
>+++ b/test/polkitbackend/test-polkitbackendjsauthority.c
>@@ -74,8 +74,8 @@ test_get_admin_identities_for_action_id (const gchar         *action_id,
> 
>   authority = get_authority ();
> 
>-  caller = polkit_unix_process_new (getpid ());
>-  subject = polkit_unix_process_new (getpid ());
>+  caller = polkit_unix_process_new_for_owner (getpid (), 0, getuid ());
>+  subject = polkit_unix_process_new_for_owner (getpid (), 0, getuid ());
>   user_for_subject = polkit_identity_from_string ("unix-user:root", &error);
>   g_assert_no_error (error);
> 
>@@ -340,8 +340,8 @@ rules_test_func (gconstpointer user_data)
> 
>   authority = get_authority ();
> 
>-  caller = polkit_unix_process_new (getpid ());
>-  subject = polkit_unix_process_new (getpid ());
>+  caller = polkit_unix_process_new_for_owner (getpid (), 0, getuid ());
>+  subject = polkit_unix_process_new_for_owner (getpid (), 0, getuid ());
>   user_for_subject = polkit_identity_from_string (tc->identity, &error);
>   g_assert_no_error (error);
> 
>-- 
>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.