proxmox-mirrors/mirror_lxc
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_lxc synced 2025-08-14 01:39:59 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #1809 from brauner/2017-09-12/start_move_env_setup

Browse Source 
start: set environment variables correctly
This commit is contained in:
Stéphane Graber 2017-09-12 19:14:27 -07:00 committed by GitHub
commit dbac6c0256
1 changed files with 18 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/lxc/start.c
View File

@ -913,16 +913,10 @@ static int do_start(void *data)
INFO("Unshared CLONE_NEWCGROUP.");
}
/* The clearenv() and putenv() calls have been moved here to allow us to
* use environment variables passed to the various hooks, such as the
* start hook above. Not all of the variables like CONFIG_PATH or ROOTFS
* are valid in this context but others are.
/* Add the requested environment variables to the current environment to
* allow them to be used by the various hooks, such as the start hook
* above.
*/
if (clearenv()) {
SYSERROR("Failed to clear environment.");
/* Don't error out though. */
}
lxc_list_for_each(iterator, &handler->conf->environment) {
if (putenv((char *)iterator->elem)) {
SYSERROR("Failed to set environment variable: %s.", (char *)iterator->elem);
@ -1005,6 +999,21 @@ static int do_start(void *data)
if (lxc_sync_barrier_parent(handler, LXC_SYNC_CGROUP_LIMITS))
goto out_warn_father;
/* Reset the environment variables the user requested in a clear
* environment.
*/
if (clearenv()) {
SYSERROR("Failed to clear environment.");
/* Don't error out though. */
}
lxc_list_for_each(iterator, &handler->conf->environment) {
if (putenv((char *)iterator->elem)) {
SYSERROR("Failed to set environment variable: %s.", (char *)iterator->elem);
goto out_warn_father;
}
}
if (putenv("container=lxc")) {
SYSERROR("Failed to set environment variable: container=lxc.");
goto out_warn_father;