Revision
23035
Author
zarzycki@apple.com
Date
2007-02-07 08:34:09 -0800 (Wed, 07 Feb 2007)

Log Message

<rdar://problem/4968241> spurious logging from the libraries used by the various importers

Modified Paths

Diff

Modified: trunk/launchd/src/launchd_core_logic.c (23034 => 23035)


--- trunk/launchd/src/launchd_core_logic.c	2007-02-06 22:10:42 UTC (rev 23034)
+++ trunk/launchd/src/launchd_core_logic.c	2007-02-07 16:34:09 UTC (rev 23035)
@@ -1803,7 +1803,7 @@
 
 	job_assumes(j, socketpair(AF_UNIX, SOCK_STREAM, 0, execspair) != -1);
 
-	if (job_assumes(j, pipe(oepair) != -1)) {
+	if (!j->legacy_mach_job && job_assumes(j, pipe(oepair) != -1)) {
 		j->log_redirect_fd = _fd(oepair[0]);
 		job_assumes(j, fcntl(j->log_redirect_fd, F_SETFL, O_NONBLOCK) != -1);
 		job_assumes(j, kevent_mod(j->log_redirect_fd, EVFILT_READ, EV_ADD, 0, 0, j) != -1);
@@ -1822,9 +1822,11 @@
 		}
 		break;
 	case 0:
-		job_assumes(j, dup2(oepair[1], STDOUT_FILENO) != -1);
-		job_assumes(j, dup2(oepair[1], STDERR_FILENO) != -1);
-		job_assumes(j, close(oepair[1]) != -1);
+		if (!j->legacy_mach_job) {
+			job_assumes(j, dup2(oepair[1], STDOUT_FILENO) != -1);
+			job_assumes(j, dup2(oepair[1], STDERR_FILENO) != -1);
+			job_assumes(j, close(oepair[1]) != -1);
+		}
 		job_assumes(j, close(execspair[0]) == 0);
 		/* wait for our parent to say they've attached a kevent to us */
 		read(_fd(execspair[1]), &c, sizeof(c));
@@ -1845,7 +1847,9 @@
 		job_start_child(j);
 		break;
 	default:
-		job_assumes(j, close(oepair[1]) != -1);
+		if (!j->legacy_mach_job) {
+			job_assumes(j, close(oepair[1]) != -1);
+		}
 		j->p = c;
 		j->forkfd = _fd(execspair[0]);
 		job_assumes(j, close(execspair[1]) == 0);