summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaul Barbieri <paul.barbieri@baesystems.com>2013-07-01 21:10:28 +0100
committerColin Walters <walters@verbum.org>2013-07-01 21:10:28 +0100
commite55ccf7338f2eb68f08d648855a4a92a091df690 (patch)
treede8df1f6b85859ed53d3b90e395960df9e657088
parent1891eafd1da6f31e6b8876b2882800a5c2b92148 (diff)
downloadglib-e55ccf7338f2eb68f08d648855a4a92a091df690.tar.gz
gspawn: Fix leak of internal pipe with G_SPAWN_LEAVE_DESCRIPTORS_OPEN
Even when the app author specifies G_SPAWN_LEAVE_DESCRIPTORS_OPEN, we should avoid leaking our internal pipe machinery into the child. Commit message written by: Colin Walters <walters@verbum.org> https://bugzilla.gnome.org/show_bug.cgi?id=703407
-rw-r--r--glib/gspawn.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/glib/gspawn.c b/glib/gspawn.c
index f9baaa381..3db3c8dfb 100644
--- a/glib/gspawn.c
+++ b/glib/gspawn.c
@@ -1395,6 +1395,7 @@ fork_exec_with_pipes (gboolean intermediate_child,
}
else if (grandchild_pid == 0)
{
+ close_and_invalidate (&child_pid_report_pipe[1]);
do_exec (child_err_report_pipe[1],
stdin_pipe[0],
stdout_pipe[1],