From: Steven Luo <steven+maemo@steven676.net>
Date: Fri, 5 Feb 2010 07:37:56 +0000 (-0800)
Subject: Throw away stdout/stderr for browserds we launch too
X-Git-Tag: v3.2~28
X-Git-Url: http://git.maemo.org/git/?p=browser-switch;a=commitdiff_plain;h=53ba5ca09766205c3bcdfdd8acf316529a758c60

Throw away stdout/stderr for browserds we launch too

Commit d8d8d1ed... ("Close stdin/stdout/stderr in child processes before
exec()") didn't prevent browserd from spewing noise, because we invoke
that via system() instead of fork()/exec().  Fix that oversight.
---

diff --git a/launcher.c b/launcher.c
index 76a5321..9c7893b 100644
--- a/launcher.c
+++ b/launcher.c
@@ -194,9 +194,9 @@ void launch_microb(struct swb_context *ctx, char *uri) {
 	if (WIFEXITED(status) && WEXITSTATUS(status)) {
 		kill_browserd = 1;
 #ifdef FREMANTLE
-		system("/usr/sbin/browserd -d -b");
+		system("/usr/sbin/browserd -d -b > /dev/null 2>&1");
 #else
-		system("/usr/sbin/browserd -d");
+		system("/usr/sbin/browserd -d > /dev/null 2>&1");
 #endif
 	}