SymbX/systemd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

pam_systemd: Let user record override env vars

Browse Source 
The user record should be the source of truth for the user's environment
variables, and the user should be able to override them in much the same
way that they can if they simply append the variable to their ~/.profile

For example, before $LANG would never get set to the user's preferred
language, because the service manager always ensures that $LANG is set
to something (either the localed config, or a compiled-in default). Thus
the user's preferredLanguage setting was always ignored
This commit is contained in:
Adrian Vovk
2024-02-13 17:31:35 -05:00
parent 6d55e3a364
commit 592ca6f0ef
1 changed files with 9 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/login/pam_systemd.c
+9 -35
View File
@@ -617,48 +617,25 @@ static int apply_user_record_settings(
}
STRV_FOREACH(i, ur->environment) {
_cleanup_free_ char *n = NULL;
const char *e;
assert_se(e = strchr(*i, '=')); /* environment was already validated while parsing JSON record, this thus must hold */
n = strndup(*i, e - *i);
if (!n)
return pam_log_oom(handle);
if (pam_getenv(handle, n)) {
pam_debug_syslog(handle, debug,
"PAM environment variable $%s already set, not changing based on record.", *i);
continue;
}
r = pam_putenv_and_log(handle, *i, debug);
if (r != PAM_SUCCESS)
return r;
}
if (ur->email_address) {
if (pam_getenv(handle, "EMAIL"))
pam_debug_syslog(handle, debug,
"PAM environment variable $EMAIL already set, not changing based on user record.");
else {
_cleanup_free_ char *joined = NULL;
_cleanup_free_ char *joined = NULL;
joined = strjoin("EMAIL=", ur->email_address);
if (!joined)
return pam_log_oom(handle);
joined = strjoin("EMAIL=", ur->email_address);
if (!joined)
return pam_log_oom(handle);
r = pam_putenv_and_log(handle, joined, debug);
if (r != PAM_SUCCESS)
return r;
}
r = pam_putenv_and_log(handle, joined, debug);
if (r != PAM_SUCCESS)
return r;
}
if (ur->time_zone) {
if (pam_getenv(handle, "TZ"))
pam_debug_syslog(handle, debug,
"PAM environment variable $TZ already set, not changing based on user record.");
else if (!timezone_is_valid(ur->time_zone, LOG_DEBUG))
if (!timezone_is_valid(ur->time_zone, LOG_DEBUG))
pam_debug_syslog(handle, debug,
"Time zone specified in user record is not valid locally, not setting $TZ.");
else {
@@ -675,10 +652,7 @@ static int apply_user_record_settings(
}
if (ur->preferred_language) {
if (pam_getenv(handle, "LANG"))
pam_debug_syslog(handle, debug,
"PAM environment variable $LANG already set, not changing based on user record.");
else if (locale_is_installed(ur->preferred_language) <= 0)
if (locale_is_installed(ur->preferred_language) <= 0)
pam_debug_syslog(handle, debug,
"Preferred language specified in user record is not valid or not installed, not setting $LANG.");
else {