Skip to content

Commit 910fee4

Browse files
syscall: append to environment in tests, don't clobber it
Fixes https://gcc.gnu.org/PR95061 Change-Id: I3cddfe5920b6da572ddeafa201617003c25175e6 Reviewed-on: https://go-review.googlesource.com/c/go/+/233318 Run-TryBot: Ian Lance Taylor <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Brad Fitzpatrick <[email protected]>
1 parent 2cb10d4 commit 910fee4

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

src/syscall/exec_linux_test.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,7 @@ func TestUnshareMountNameSpace(t *testing.T) {
355355
}
356356

357357
cmd := exec.Command(os.Args[0], "-test.run=TestUnshareMountNameSpaceHelper", d)
358-
cmd.Env = []string{"GO_WANT_HELPER_PROCESS=1"}
358+
cmd.Env = append(os.Environ(), "GO_WANT_HELPER_PROCESS=1")
359359
cmd.SysProcAttr = &syscall.SysProcAttr{Unshareflags: syscall.CLONE_NEWNS}
360360

361361
o, err := cmd.CombinedOutput()
@@ -406,7 +406,7 @@ func TestUnshareMountNameSpaceChroot(t *testing.T) {
406406
}
407407

408408
cmd = exec.Command("/syscall.test", "-test.run=TestUnshareMountNameSpaceHelper", "/")
409-
cmd.Env = []string{"GO_WANT_HELPER_PROCESS=1"}
409+
cmd.Env = append(os.Environ(), "GO_WANT_HELPER_PROCESS=1")
410410
cmd.SysProcAttr = &syscall.SysProcAttr{Chroot: d, Unshareflags: syscall.CLONE_NEWNS}
411411

412412
o, err := cmd.CombinedOutput()
@@ -621,7 +621,7 @@ func testAmbientCaps(t *testing.T, userns bool) {
621621
}
622622

623623
cmd := exec.Command(f.Name(), "-test.run=TestAmbientCapsHelper")
624-
cmd.Env = []string{"GO_WANT_HELPER_PROCESS=1"}
624+
cmd.Env = append(os.Environ(), "GO_WANT_HELPER_PROCESS=1")
625625
cmd.Stdout = os.Stdout
626626
cmd.Stderr = os.Stderr
627627
cmd.SysProcAttr = &syscall.SysProcAttr{

src/syscall/syscall_linux_test.go

+6-3
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ func TestLinuxDeathSignal(t *testing.T) {
187187
}
188188

189189
cmd := exec.Command(tmpBinary)
190-
cmd.Env = []string{"GO_DEATHSIG_PARENT=1"}
190+
cmd.Env = append(os.Environ(), "GO_DEATHSIG_PARENT=1")
191191
chldStdin, err := cmd.StdinPipe()
192192
if err != nil {
193193
t.Fatalf("failed to create new stdin pipe: %v", err)
@@ -225,7 +225,10 @@ func TestLinuxDeathSignal(t *testing.T) {
225225

226226
func deathSignalParent() {
227227
cmd := exec.Command(os.Args[0])
228-
cmd.Env = []string{"GO_DEATHSIG_CHILD=1"}
228+
cmd.Env = append(os.Environ(),
229+
"GO_DEATHSIG_PARENT=",
230+
"GO_DEATHSIG_CHILD=1",
231+
)
229232
cmd.Stdin = os.Stdin
230233
cmd.Stdout = os.Stdout
231234
attrs := syscall.SysProcAttr{
@@ -356,7 +359,7 @@ func TestSyscallNoError(t *testing.T) {
356359
}
357360

358361
cmd := exec.Command(tmpBinary)
359-
cmd.Env = []string{"GO_SYSCALL_NOERROR=1"}
362+
cmd.Env = append(os.Environ(), "GO_SYSCALL_NOERROR=1")
360363

361364
out, err := cmd.CombinedOutput()
362365
if err != nil {

0 commit comments

Comments
 (0)