libgo patch committed: Make sure file is not closed early

Submitted by Ian Taylor on Feb. 1, 2012, 9:02 p.m.


Message ID
State New
Headers show

Commit Message

Ian Taylor Feb. 1, 2012, 9:02 p.m.
This patch to libgo fixes the test for leaked file descriptors.  In some
cases the file, expected to be open, could be collected by the garbage
collector and closed.  This then caused the next call to open to return
that descriptor rather than the expected one, causing the test to fail.
This patch adds an explicit close, ensuring that the file can not be
collected.  Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu.  Committed to mainline.


Patch hide | download patch | download mbox

Index: libgo/go/os/exec/exec_test.go
--- libgo/go/os/exec/exec_test.go	(revision 183650)
+++ libgo/go/os/exec/exec_test.go	(working copy)
@@ -292,6 +292,7 @@  func TestHelperProcess(*testing.T) {
+		fd3.Close()
 	case "exit":
 		n, _ := strconv.Atoi(args[0])