Repository: mesos Updated Branches: refs/heads/master d2a1d8c53 -> f32c1ba64
Fixed a file descriptor leak in SubprocessTest.PipeRedirect. Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/f32c1ba6 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/f32c1ba6 Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/f32c1ba6 Branch: refs/heads/master Commit: f32c1ba6456d6948572935936640371bc7d1308a Parents: d2a1d8c Author: Benjamin Mahler <[email protected]> Authored: Sat Dec 12 10:49:19 2015 -0800 Committer: Benjamin Mahler <[email protected]> Committed: Sat Dec 12 14:41:07 2015 -0800 ---------------------------------------------------------------------- 3rdparty/libprocess/src/tests/subprocess_tests.cpp | 3 +++ 1 file changed, 3 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/f32c1ba6/3rdparty/libprocess/src/tests/subprocess_tests.cpp ---------------------------------------------------------------------- diff --git a/3rdparty/libprocess/src/tests/subprocess_tests.cpp b/3rdparty/libprocess/src/tests/subprocess_tests.cpp index d1e51b4..869b920 100644 --- a/3rdparty/libprocess/src/tests/subprocess_tests.cpp +++ b/3rdparty/libprocess/src/tests/subprocess_tests.cpp @@ -251,6 +251,9 @@ TEST_F(SubprocessTest, PipeRedirect) ASSERT_SOME(os::nonblock(s.get().out().get())); AWAIT_READY(io::redirect(s.get().out().get(), fd.get())); + // Close our copy of the fd. + EXPECT_SOME(os::close(fd.get())); + // Advance time until the internal reaper reaps the subprocess. Clock::pause(); while (s.get().status().isPending()) {
