From 01d8d43d0b66f349a38075a3ebfd82fc9bed084a Mon Sep 17 00:00:00 2001
From: Colin Walters <walters@verbum.org>
Date: Thu, 21 Jun 2012 15:37:39 -0400
Subject: [PATCH] gspawn: Abort if we can't open /dev/null

This really "shouldn't happen", but if we have an assertion here, it
will help static analysis tools know we're not hitting undefined
state.
---
 glib/gspawn.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/glib/gspawn.c b/glib/gspawn.c
index bdecff20b..5d8d57b5b 100644
--- a/glib/gspawn.c
+++ b/glib/gspawn.c
@@ -1099,6 +1099,7 @@ do_exec (gint                  child_err_report_fd,
     {
       /* Keep process from blocking on a read of stdin */
       gint read_null = open ("/dev/null", O_RDONLY);
+      g_assert (read_null != -1);
       sane_dup2 (read_null, 0);
       close_and_invalidate (&read_null);
     }
@@ -1117,6 +1118,7 @@ do_exec (gint                  child_err_report_fd,
   else if (stdout_to_null)
     {
       gint write_null = sane_open ("/dev/null", O_WRONLY);
+      g_assert (write_null != -1);
       sane_dup2 (write_null, 1);
       close_and_invalidate (&write_null);
     }