Attachment 76324 Details for Bug 62016 – pkexec: Set process environment from pam_getenvlist()

[patch] pkexec: Set process environment from pam_getenvlist()

0001-pkexec-Set-process-environment-from-pam_getenvlist.patch (text/plain), 1.18 KB, created by Martin Pitt on 2013-03-11 08:48:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Pitt

Created: 2013-03-11 08:48:38 UTC

Size: 1.18 KB

patch

obsolete

>From adf59cd6d55a53be7a0242da874622cc7289c616 Mon Sep 17 00:00:00 2001
>From: Steve Langasek <steve.langasek@canonical.com>
>Date: Fri, 8 Mar 2013 12:00:00 +0100
>Subject: [PATCH] pkexec: Set process environment from pam_getenvlist()
>
>Various pam modules provide environment variables that are intended to be set
>in the environment of the pam session.  pkexec needs to process the output of
>pam_getenvlist() to get these.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=62016
>---
> src/programs/pkexec.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
>diff --git a/src/programs/pkexec.c b/src/programs/pkexec.c
>index f42a3bb..529898a 100644
>--- a/src/programs/pkexec.c
>+++ b/src/programs/pkexec.c
>@@ -145,6 +145,7 @@ open_session (const gchar *user_to_auth)
>   gboolean ret;
>   gint rc;
>   pam_handle_t *pam_h;
>+  char **envlist;
>   struct pam_conv conversation;
> 
>   ret = FALSE;
>@@ -176,6 +177,15 @@ open_session (const gchar *user_to_auth)
> 
>   ret = TRUE;
> 
>+  envlist = pam_getenvlist (pam_h);
>+  if (envlist != NULL)
>+    {
>+      guint n;
>+      for (n = 0; envlist[n]; n++)
>+        putenv (envlist[n]);
>+      free (envlist);
>+    }
>+
> out:
>   if (pam_h != NULL)
>     pam_end (pam_h, rc);
>-- 
>1.8.1.2
>

Actions: View | Splinter Review

Attachments on bug 62016: 76150 | 76324

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.