Skip to content

Commit 42dd8bf

Browse files
blink-so[bot]f0ssel
andcommitted
fix: add LOGNAME to environment restoration
Some tools check LOGNAME instead of USER to determine the current user. Adding LOGNAME=original_user to ensure consistent user identity across all environment variables. Co-authored-by: f0ssel <[email protected]>
1 parent 463dbe8 commit 42dd8bf

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

network/linux.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,8 @@ func (l *LinuxJail) Execute(command []string, extraEnv map[string]string) error
106106
env = append(env, fmt.Sprintf("HOME=%s", user.HomeDir))
107107
// Set USER to original username
108108
env = append(env, fmt.Sprintf("USER=%s", sudoUser))
109+
// Set LOGNAME to original username (some tools check this instead of USER)
110+
env = append(env, fmt.Sprintf("LOGNAME=%s", sudoUser))
109111
l.logger.Debug("Restored user environment", "home", user.HomeDir, "user", sudoUser)
110112
}
111113
}

network/macos.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,8 @@ func (m *MacOSNetJail) Execute(command []string, extraEnv map[string]string) err
9292
env = append(env, fmt.Sprintf("HOME=%s", user.HomeDir))
9393
// Set USER to original username
9494
env = append(env, fmt.Sprintf("USER=%s", sudoUser))
95+
// Set LOGNAME to original username (some tools check this instead of USER)
96+
env = append(env, fmt.Sprintf("LOGNAME=%s", sudoUser))
9597
m.logger.Debug("Restored user environment", "home", user.HomeDir, "user", sudoUser)
9698
}
9799
}

0 commit comments

Comments
 (0)