github/xrdp
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #3695 from akarl10/user-shell-environment-fix

Browse Source 
User shell environment fix
This commit is contained in:
matt335672
2026-02-12 11:26:52 +00:00
committed by GitHub
parent eb4514210d c1ed8b8eb6
commit b3ab2c28d8
1 changed files with 26 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files
sesman/sesexec/session.c
+8 -6
View File
@@ -257,9 +257,12 @@ start_window_manager(const struct login_info *login_info,
}
}
if (g_cfg->sec.allow_alternate_shell &&
g_cfg->sec.pass_shell_as_env != NULL &&
g_cfg->sec.pass_shell_as_env[0] != '0')
if (s->shell[0] != '\0')
{
if (g_cfg->sec.allow_alternate_shell)
{
if (g_cfg->sec.pass_shell_as_env != NULL &&
g_cfg->sec.pass_shell_as_env[0] != '\0')
{
// Pass the shell in to the standard startwm scripts
// in an environment variable
@@ -269,11 +272,9 @@ start_window_manager(const struct login_info *login_info,
s->shell);
g_setenv_log(g_cfg->sec.pass_shell_as_env, s->shell, 1);
}
else if (s->shell[0] != '\0')
else
{
// Try to execute the shell directly (if permitted)
if (g_cfg->sec.allow_alternate_shell)
{
if (g_strchr(s->shell, ' ') != 0 || g_strchr(s->shell, '\t') != 0)
{
LOG(LOG_LEVEL_INFO,
@@ -291,6 +292,7 @@ start_window_manager(const struct login_info *login_info,
g_execlp3(s->shell, s->shell, 0);
}
}
}
else
{
LOG(LOG_LEVEL_WARNING,