[launchd-changes] [23035] trunk/launchd/src/launchd_core_logic.c

source_changes at macosforge.org source_changes at macosforge.org
Wed Feb 7 08:34:09 PST 2007


Revision: 23035
          http://trac.macosforge.org/projects/launchd/changeset/23035
Author:   zarzycki at 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:
--------------
    trunk/launchd/src/launchd_core_logic.c

Modified: trunk/launchd/src/launchd_core_logic.c
===================================================================
--- 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);

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.macosforge.org/pipermail/launchd-changes/attachments/20070207/0bcd87af/attachment.html


More information about the launchd-changes mailing list