This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
libgo patch committed: Make sure file is not closed early
- From: Ian Lance Taylor <iant at google dot com>
- To: gcc-patches at gcc dot gnu dot org, gofrontend-dev at googlegroups dot com
- Date: Wed, 01 Feb 2012 13:02:11 -0800
- Subject: libgo patch committed: Make sure file is not closed early
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.
Ian
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) {
f.Close()
}
}
+ fd3.Close()
os.Stderr.Write(bs)
case "exit":
n, _ := strconv.Atoi(args[0])