From 9b830f613dccfad87758ba8987d8f0d67187426b Mon Sep 17 00:00:00 2001
From: Michael Crosby <michael@crosbymichael.com>
Date: Wed, 4 Jun 2014 17:54:00 -0700
Subject: [PATCH] Move env load to nsenter Docker-DCO-1.1-Signed-off-by:
 Michael Crosby <michael@crosbymichael.com> (github: crosbymichael)

---
 libcontainer/namespaces/execin.go | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/libcontainer/namespaces/execin.go b/libcontainer/namespaces/execin.go
index 699c67d..4d5671e 100644
--- a/libcontainer/namespaces/execin.go
+++ b/libcontainer/namespaces/execin.go
@@ -14,12 +14,6 @@ import (
 
 // ExecIn uses an existing pid and joins the pid's namespaces with the new command.
 func ExecIn(container *libcontainer.Container, nspid int, args []string) error {
-	// clear the current processes env and replace it with the environment
-	// defined on the container
-	if err := LoadContainerEnvironment(container); err != nil {
-		return err
-	}
-
 	// TODO(vmarmol): If this gets too long, send it over a pipe to the child.
 	// Marshall the container into JSON since it won't be available in the namespace.
 	containerJson, err := json.Marshal(container)
@@ -36,7 +30,7 @@ func ExecIn(container *libcontainer.Container, nspid int, args []string) error {
 	// Enter the namespace and then finish setup
 	finalArgs := []string{os.Args[0], "nsenter", strconv.Itoa(nspid), processLabel, string(containerJson)}
 	finalArgs = append(finalArgs, args...)
-	if err := system.Execv(finalArgs[0], finalArgs[0:], container.Env); err != nil {
+	if err := system.Execv(finalArgs[0], finalArgs[0:], os.Environ()); err != nil {
 		return err
 	}
 	panic("unreachable")
@@ -44,13 +38,18 @@ func ExecIn(container *libcontainer.Container, nspid int, args []string) error {
 
 // NsEnter is run after entering the namespace.
 func NsEnter(container *libcontainer.Container, processLabel string, nspid int, args []string) error {
+	// clear the current processes env and replace it with the environment
+	// defined on the container
+	if err := LoadContainerEnvironment(container); err != nil {
+		return err
+	}
 	if err := FinalizeNamespace(container); err != nil {
 		return err
 	}
 	if err := label.SetProcessLabel(processLabel); err != nil {
 		return err
 	}
-	if err := system.Execv(args[0], args[0:], os.Environ()); err != nil {
+	if err := system.Execv(args[0], args[0:], container.Env); err != nil {
 		return err
 	}
 	panic("unreachable")