diff options
author | Stef Walter <stefw@gnome.org> | 2014-03-14 11:07:09 +0100 |
---|---|---|
committer | Stef Walter <stefw@gnome.org> | 2014-03-14 11:30:38 +0100 |
commit | deae9b5b34d35acc62cd045dcf187e40b8c46200 (patch) | |
tree | 49a72cfede04ee0c1dbc237fb9611a7cc1714bec | |
parent | ce8b568934fcb621f7ffaef18d55f0218d7b76d2 (diff) |
pam: Pass XDG_RUNTIME_DIR to new process
If XDG_RUNTIME_DIR is not in the PAM envlist, but *is* in the
process environment, then steal it from there similar to how
we handle DISPLAY.
https://bugzilla.gnome.org/show_bug.cgi?id=726196
-rw-r--r-- | pam/gkr-pam-module.c | 8 | ||||
-rw-r--r-- | pam/test-pam.c | 9 |
2 files changed, 16 insertions, 1 deletions
diff --git a/pam/gkr-pam-module.c b/pam/gkr-pam-module.c index c06d8321..35927318 100644 --- a/pam/gkr-pam-module.c +++ b/pam/gkr-pam-module.c @@ -348,6 +348,7 @@ setup_child (int inp[2], const char *argument) { const char* display; + const char *runtime; int i, ret; char *args[] = { @@ -404,7 +405,12 @@ setup_child (int inp[2], if (display) ret = setup_pam_env (ph, "DISPLAY", display); } - + if (ret == PAM_SUCCESS && !pam_getenv (ph, "XDG_RUNTIME_DIR")) { + runtime = getenv ("XDG_RUNTIME_DIR"); + if (runtime) + ret = setup_pam_env (ph, "XDG_RUNTIME_DIR", runtime); + } + /* Make sure that worked */ if (ret != PAM_SUCCESS) { syslog (GKR_LOG_ERR, "gkr-pam: couldn't setup environment: %s", diff --git a/pam/test-pam.c b/pam/test-pam.c index 56eebcae..55f49ac6 100644 --- a/pam/test-pam.c +++ b/pam/test-pam.c @@ -278,6 +278,9 @@ test_starts_creates (Test *test, if (test->skipping) return; + /* We're testing that we create the directory appropriately */ + g_unsetenv ("XDG_RUNTIME_DIR"); + start_in_session = (strstr (pam_conf, "session") != NULL); login_keyring = g_build_filename (test->directory, "login.keyring", NULL); @@ -322,6 +325,9 @@ test_starts_only_session (Test *test, /* This is the PAM config that starts the daemon from session handler */ g_assert (strstr (pam_conf, "session-start") != NULL); + /* We're testing that we create the directory appropriately */ + g_unsetenv ("XDG_RUNTIME_DIR"); + login_keyring = g_build_filename (test->directory, "login.keyring", NULL); g_assert (!g_file_test (login_keyring, G_FILE_TEST_EXISTS)); @@ -345,6 +351,9 @@ test_starts_exists (Test *test, if (test->skipping) return; + /* We're testing that we create the directory appropriately */ + g_unsetenv ("XDG_RUNTIME_DIR"); + start_in_session = (strstr (pam_conf, "session") != NULL); egg_tests_copy_scratch_file (test->directory, SRCDIR "/pam/fixtures/login.keyring"); |