From 2261adca68ec0924651b957c7ce17dce6590e43e Mon Sep 17 00:00:00 2001 From: Dale Tristram Date: Tue, 7 Aug 2018 19:09:49 +0200 Subject: [PATCH] Ensure cmd.SysProcAttr is set before modifying it (#275) --- 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 3b8ca29cc..191e3d638 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")