From f12bfb5f38fe90c5f666513777b0daa5819672f8 Mon Sep 17 00:00:00 2001 From: Dale Tristram Date: Fri, 3 Aug 2018 22:37:37 +0200 Subject: [PATCH] Ensure cmd.SysProcAttr is set before modifying it --- pkg/commands/run.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/commands/run.go b/pkg/commands/run.go index 3b8ca29cce..191e3d6386 100644 --- a/pkg/commands/run.go +++ b/pkg/commands/run.go @@ -60,6 +60,7 @@ func (r *RunCommand) ExecuteCommand(config *v1.Config, buildArgs *dockerfile.Bui cmd.Stderr = os.Stderr replacementEnvs := buildArgs.ReplacementEnvs(config.Env) cmd.Env = replacementEnvs + cmd.SysProcAttr = &syscall.SysProcAttr{Setpgid: true} // If specified, run the command as a specific user if config.User != "" { @@ -91,7 +92,6 @@ func (r *RunCommand) ExecuteCommand(config *v1.Config, buildArgs *dockerfile.Bui } cmd.SysProcAttr.Credential = &syscall.Credential{Uid: uid, Gid: gid} } - cmd.SysProcAttr = &syscall.SysProcAttr{Setpgid: true} if err := cmd.Start(); err != nil { return errors.Wrap(err, "starting command")