December 2006 - launchd-changes - lists.macosforge.org

[22997] tags/launchd-184/
by source_changes＠macosforge.org 22 Dec '06

22 Dec '06

Revision: 22997 http://trac.macosforge.org/projects/launchd/changeset/22997 Author: zarzycki(a)apple.com Date: 2006-12-22 11:44:52 -0800 (Fri, 22 Dec 2006) Log Message: ----------- "Tagging launchd-184 from https://svn.macosforge.org/repository/launchd/trunk" Added Paths: ----------- tags/launchd-184/ Copied: tags/launchd-184 (from rev 22996, trunk)

1 0

[22996] trunk/launchd/src
by source_changes＠macosforge.org 22 Dec '06

22 Dec '06

Revision: 22996 http://trac.macosforge.org/projects/launchd/changeset/22996 Author: zarzycki(a)apple.com Date: 2006-12-22 11:05:56 -0800 (Fri, 22 Dec 2006) Log Message: ----------- <rdar://problem/4802454> CAS: WindowServer hangs in CGXProcessHIDEvent/.../CGXPseudoActivateWindow/.../_LSDoSetFrontProcess Modified Paths: -------------- trunk/launchd/src/launchd_core_logic.c trunk/launchd/src/libvproc.c Modified: trunk/launchd/src/launchd_core_logic.c =================================================================== --- trunk/launchd/src/launchd_core_logic.c 2006-12-21 19:43:50 UTC (rev 22995) +++ trunk/launchd/src/launchd_core_logic.c 2006-12-22 19:05:56 UTC (rev 22996) @@ -748,6 +748,11 @@ goto out_bad2; } + if (!job_assumes(j, launchd_mport_notify_req(j->j_port, MACH_NOTIFY_NO_SENDERS) == KERN_SUCCESS)) { + job_assumes(j, launchd_mport_close_recv(j->j_port) == KERN_SUCCESS); + goto out_bad; + } + return true; out_bad2: job_assumes(j, launchd_mport_close_recv(j->j_port) == KERN_SUCCESS); @@ -788,11 +793,6 @@ goto out_bad; } - if (!job_assumes(jr, launchd_mport_notify_req(jr->j_port, MACH_NOTIFY_NO_SENDERS) == KERN_SUCCESS)) { - job_assumes(jr, launchd_mport_close_recv(jr->j_port) == KERN_SUCCESS); - goto out_bad; - } - job_log(jr, LOG_INFO, "Legacy%s server created", ond ? " on-demand" : ""); return jr; @@ -2691,9 +2691,12 @@ bool job_useless(job_t j) { - /* Yes, j->unload_at_exit and j->j->only_once seem the same, but they'll differ someday... */ + /* Yes, j->unload_at_exit and j->only_once seem the same, but they'll differ someday... */ if ((j->unload_at_exit || j->only_once) && j->start_time != 0) { + if (j->unload_at_exit && j->j_port) { + return false; + } job_log(j, LOG_INFO, "Exited. Was only configured to run once."); return true; } else if (shutdown_in_progress) { Modified: trunk/launchd/src/libvproc.c =================================================================== --- trunk/launchd/src/libvproc.c 2006-12-21 19:43:50 UTC (rev 22995) +++ trunk/launchd/src/libvproc.c 2006-12-22 19:05:56 UTC (rev 22996) @@ -170,22 +170,11 @@ return -1; } -#if 0 kern_return_t mpm_wait(mach_port_t ajob __attribute__((unused)), int *wstatus) { - /* Stubbed out for now... - * <rdar://problem/4802454> CAS: WindowServer hangs in CGXProcessHIDEvent/.../CGXPseudoActivateWindow/.../_LSDoSetFrontProcess - */ return vproc_mig_wait(ajob, wstatus); } -#else -kern_return_t -mpm_wait(mach_port_t ajob __attribute__((unused)), int *wstatus __attribute__((unused))) -{ - return MIG_BAD_ID; -} -#endif kern_return_t mpm_uncork_fork(mach_port_t ajob)

1 0

[22995] trunk/launchd/src/launchd_core_logic.c
by source_changes＠macosforge.org 21 Dec '06

21 Dec '06

Revision: 22995 http://trac.macosforge.org/projects/launchd/changeset/22995 Author: zarzycki(a)apple.com Date: 2006-12-21 11:43:50 -0800 (Thu, 21 Dec 2006) Log Message: ----------- <rdar://problem/4894737> launchd support needed for Crash Reporter to handle 64 bit exception codes Modified Paths: -------------- trunk/launchd/src/launchd_core_logic.c Modified: trunk/launchd/src/launchd_core_logic.c =================================================================== --- trunk/launchd/src/launchd_core_logic.c 2006-12-21 19:23:38 UTC (rev 22994) +++ trunk/launchd/src/launchd_core_logic.c 2006-12-21 19:43:50 UTC (rev 22995) @@ -2983,6 +2983,14 @@ job_assumes(ms->job, host_set_UNDServer(mhp, ms->port) == KERN_SUCCESS); } break; + case LAUNCH_DATA_DICTIONARY: +#ifdef MACH_EXCEPTION_CODES + if (strcasecmp(key, LAUNCH_JOBKEY_MACH_EXCEPTIONSERVER) == 0) { + job_assumes(ms->job, task_set_exception_ports(mts, em, ms->port, + EXCEPTION_STATE_IDENTITY | MACH_EXCEPTION_CODES, f) == KERN_SUCCESS); + } +#endif + break; default: break; }

1 0

[22994] trunk/launchd/src
by source_changes＠macosforge.org 21 Dec '06

21 Dec '06

Revision: 22994 http://trac.macosforge.org/projects/launchd/changeset/22994 Author: zarzycki(a)apple.com Date: 2006-12-21 11:23:38 -0800 (Thu, 21 Dec 2006) Log Message: ----------- <rdar://problem/4890178> ER: Specific binary slice selection for spawn_via_launchd Modified Paths: -------------- trunk/launchd/src/launchd_core_logic.c trunk/launchd/src/launchd_mig_types.defs trunk/launchd/src/liblaunch_private.h trunk/launchd/src/liblaunch_public.h trunk/launchd/src/libvproc.c trunk/launchd/src/libvproc_internal.h trunk/launchd/src/protocol_job.defs Modified: trunk/launchd/src/launchd_core_logic.c =================================================================== --- trunk/launchd/src/launchd_core_logic.c 2006-12-20 00:48:11 UTC (rev 22993) +++ trunk/launchd/src/launchd_core_logic.c 2006-12-21 19:23:38 UTC (rev 22994) @@ -238,6 +238,8 @@ SLIST_HEAD(, machservice) machservices; SLIST_HEAD(, semaphoreitem) semaphores; struct rusage ru; + binpref_t j_binpref; + size_t j_binpref_cnt; mach_port_t j_port; mach_port_t wait_reply_port; uid_t mach_uid; @@ -266,8 +268,8 @@ importing_global_env:1, importing_hard_limits:1, setmask:1, legacy_mach_job:1, runatload:1, anonymous:1; mode_t mask; - unsigned int globargv:1, wait4debugger:1, unload_at_exit:1, force_ppc:1, stall_before_exec:1, - only_once:1, currently_ignored:1, forced_peers_to_demand_mode:1; + unsigned int globargv:1, wait4debugger:1, unload_at_exit:1, stall_before_exec:1, only_once:1, + currently_ignored:1, forced_peers_to_demand_mode:1; char label[0]; }; @@ -297,7 +299,7 @@ static void job_reparent_to_aqua_hack(job_t j); static void job_callback(void *obj, struct kevent *kev); static launch_data_t job_export2(job_t j, bool subjobs); -static job_t job_new_spawn(job_t j, const char *label, const char *path, const char *workingdir, const char *const *argv, const char *const *env, mode_t *u_mask, bool w4d, bool fppc); +static job_t job_new_spawn(job_t j, const char *label, const char *path, const char *workingdir, const char *const *argv, const char *const *env, mode_t *u_mask, bool w4d); static job_t job_new_via_mach_init(job_t j, const char *cmd, uid_t uid, bool ond); static const char *job_prog(job_t j); static pid_t job_get_pid(job_t j); @@ -816,7 +818,7 @@ } job_t -job_new_spawn(job_t j, const char *label, const char *path, const char *workingdir, const char *const *argv, const char *const *env, mode_t *u_mask, bool w4d, bool fppc) +job_new_spawn(job_t j, const char *label, const char *path, const char *workingdir, const char *const *argv, const char *const *env, mode_t *u_mask, bool w4d) { job_t jr; size_t i; @@ -843,7 +845,6 @@ jr->unload_at_exit = true; jr->stall_before_exec = w4d; - jr->force_ppc = fppc; if (workingdir) { jr->workingdir = strdup(workingdir); @@ -870,8 +871,6 @@ envitem_new(jr, tmpstr, eqoff + 1, false); } - job_dispatch(jr, true); - return jr; } @@ -1018,12 +1017,6 @@ job_import_bool(job_t j, const char *key, bool value) { switch (key[0]) { - case 'f': - case 'F': - if (strcasecmp(key, LAUNCH_JOBKEY_FORCEPOWERPC) == 0) { - j->force_ppc = value; - } - break; case 'k': case 'K': if (strcasecmp(key, LAUNCH_JOBKEY_KEEPALIVE) == 0) { @@ -1838,9 +1831,10 @@ pid_t junk_pid; glob_t g; short spflags = POSIX_SPAWN_SETEXEC; + size_t binpref_out_cnt = 0; int i; - posix_spawnattr_init(&spattr); + job_assumes(j, posix_spawnattr_init(&spattr) == 0); job_setup_attributes(j); @@ -1878,17 +1872,13 @@ spflags |= POSIX_SPAWN_START_SUSPENDED; } - if (j->force_ppc) { - int affinmib[] = { CTL_KERN, KERN_AFFINITY, 1, 1 }; - size_t mibsz = sizeof(affinmib) / sizeof(affinmib[0]); + job_assumes(j, posix_spawnattr_setflags(&spattr, spflags) == 0); - if (sysctl(affinmib, mibsz, NULL, NULL, NULL, 0) == -1) { - job_log_error(j, LOG_WARNING, "Failed to force PowerPC execution"); - } + if (j->j_binpref_cnt) { + job_assumes(j, posix_spawnattr_setbinpref_np(&spattr, j->j_binpref_cnt, j->j_binpref, &binpref_out_cnt) == 0); + job_assumes(j, binpref_out_cnt == j->j_binpref_cnt); } - posix_spawnattr_setflags(&spattr, spflags); - if (j->prog) { posix_spawn(&junk_pid, j->inetcompat ? file2exec : j->prog, NULL, &spattr, (char *const*)argv, environ); job_log_error(j, LOG_ERR, "posix_spawn(\"%s\", ...)", j->prog); @@ -4352,7 +4342,7 @@ kern_return_t job_mig_spawn(job_t j, _internal_string_t charbuf, mach_msg_type_number_t charbuf_cnt, uint32_t argc, uint32_t envc, uint64_t flags, uint16_t mig_umask, - pid_t *child_pid, mach_port_t *obsvr_port) + binpref_t bin_pref, uint32_t binpref_cnt, pid_t *child_pid, mach_port_t *obsvr_port) { job_t jr; size_t offset = 0; @@ -4406,7 +4396,7 @@ } jr = job_new_spawn(j, label, path, workingdir, argv, env, flags & SPAWN_HAS_UMASK ? &mig_umask : NULL, - flags & SPAWN_WANTS_WAIT4DEBUGGER, flags & SPAWN_WANTS_FORCE_PPC); + flags & SPAWN_WANTS_WAIT4DEBUGGER); if (jr == NULL) switch (errno) { case EEXIST: @@ -4415,14 +4405,17 @@ return BOOTSTRAP_NO_MEMORY; } + memcpy(jr->j_binpref, bin_pref, sizeof(jr->j_binpref)); + jr->j_binpref_cnt = binpref_cnt; + + job_dispatch(jr, true); + if (!job_setup_machport(jr)) { job_remove(jr); return BOOTSTRAP_NO_MEMORY; } - job_log(j, LOG_INFO, "Spawned with flags:%s%s", - flags & SPAWN_WANTS_FORCE_PPC ? " ppc": "", - flags & SPAWN_WANTS_WAIT4DEBUGGER ? " stopped": ""); + job_log(j, LOG_INFO, "Spawned with flags:%s", flags & SPAWN_WANTS_WAIT4DEBUGGER ? " stopped": ""); *child_pid = job_get_pid(jr); *obsvr_port = jr->j_port; Modified: trunk/launchd/src/launchd_mig_types.defs =================================================================== --- trunk/launchd/src/launchd_mig_types.defs 2006-12-20 00:48:11 UTC (rev 22993) +++ trunk/launchd/src/launchd_mig_types.defs 2006-12-21 19:23:38 UTC (rev 22994) @@ -35,6 +35,7 @@ type _internal_string_t = ^array [] of char; type bootstrap_status_t = integer_t; type bootstrap_status_array_t = ^array [] of bootstrap_status_t; +type binpref_t = array [8] of integer_t; type job_t = mach_port_t intran: job_t job_mig_intran(mach_port_t) Modified: trunk/launchd/src/liblaunch_private.h =================================================================== --- trunk/launchd/src/liblaunch_private.h 2006-12-20 00:48:11 UTC (rev 22993) +++ trunk/launchd/src/liblaunch_private.h 2006-12-21 19:23:38 UTC (rev 22994) @@ -20,6 +20,7 @@ #ifndef _LAUNCH_PRIV_H_ #define _LAUNCH_PRIV_H_ +#include <mach/mach.h> #include <sys/types.h> #include <launch.h> @@ -93,7 +94,6 @@ */ #define SPAWN_VIA_LAUNCHD_STOPPED 0x0001 -#define SPAWN_VIA_LAUNCHD_FORCE_PPC 0x0002 struct spawn_via_launchd_attr { uint64_t spawn_flags; @@ -102,9 +102,11 @@ const char *const * spawn_env; const mode_t * spawn_umask; mach_port_t * spawn_observer_port; + const cpu_type_t * spawn_binpref; + size_t spawn_binpref_cnt; }; -#define spawn_via_launchd(a, b, c) _spawn_via_launchd(a, b, c, 0) +#define spawn_via_launchd(a, b, c) _spawn_via_launchd(a, b, c, 1) pid_t _spawn_via_launchd( const char *label, const char *const *argv, Modified: trunk/launchd/src/liblaunch_public.h =================================================================== --- trunk/launchd/src/liblaunch_public.h 2006-12-20 00:48:11 UTC (rev 22993) +++ trunk/launchd/src/liblaunch_public.h 2006-12-21 19:23:38 UTC (rev 22994) @@ -75,7 +75,6 @@ #define LAUNCH_JOBKEY_WORKINGDIRECTORY "WorkingDirectory" #define LAUNCH_JOBKEY_ENVIRONMENTVARIABLES "EnvironmentVariables" #define LAUNCH_JOBKEY_USERENVIRONMENTVARIABLES "UserEnvironmentVariables" -#define LAUNCH_JOBKEY_FORCEPOWERPC "ForcePowerPC" #define LAUNCH_JOBKEY_UMASK "Umask" #define LAUNCH_JOBKEY_NICE "Nice" #define LAUNCH_JOBKEY_LOWPRIORITYIO "LowPriorityIO" Modified: trunk/launchd/src/libvproc.c =================================================================== --- trunk/launchd/src/libvproc.c 2006-12-20 00:48:11 UTC (rev 22993) +++ trunk/launchd/src/libvproc.c 2006-12-21 19:23:38 UTC (rev 22994) @@ -68,10 +68,14 @@ uint64_t flags = 0; uint32_t argc = 0; uint32_t envc = 0; + binpref_t bin_pref; + size_t binpref_cnt = 0, binpref_max = sizeof(bin_pref) / sizeof(bin_pref[0]); pid_t p = -1; mode_t u_mask = CMASK; mach_port_t obsvr_port = MACH_PORT_NULL; + memset(&bin_pref, 0, sizeof(bin_pref)); + for (tmpp = argv; *tmpp; tmpp++) { argc++; len = strlen(*tmpp) + 1; @@ -81,13 +85,21 @@ } if (spawn_attrs) switch (struct_version) { + case 1: + if (spawn_attrs->spawn_binpref) { + if (spawn_attrs->spawn_binpref_cnt < binpref_max) { + binpref_max = spawn_attrs->spawn_binpref_cnt; + } + + for (; binpref_cnt < binpref_max; binpref_cnt++) { + bin_pref[binpref_cnt] = spawn_attrs->spawn_binpref[binpref_cnt]; + } + } + case 0: if (spawn_attrs->spawn_flags & SPAWN_VIA_LAUNCHD_STOPPED) { flags |= SPAWN_WANTS_WAIT4DEBUGGER; } - if (spawn_attrs->spawn_flags & SPAWN_VIA_LAUNCHD_FORCE_PPC) { - flags |= SPAWN_WANTS_FORCE_PPC; - } if (spawn_attrs->spawn_env) { for (tmpp = spawn_attrs->spawn_env; *tmpp; tmpp++) { @@ -125,13 +137,13 @@ break; } - kr = vproc_mig_spawn(bootstrap_port, buf, buf_len, argc, envc, flags, u_mask, &p, &obsvr_port); + kr = vproc_mig_spawn(bootstrap_port, buf, buf_len, argc, envc, flags, u_mask, bin_pref, binpref_cnt, &p, &obsvr_port); if (kr == VPROC_ERR_TRY_PER_USER) { mach_port_t puc; if (vproc_mig_lookup_per_user_context(bootstrap_port, 0, &puc) == 0) { - kr = vproc_mig_spawn(puc, buf, buf_len, argc, envc, flags, u_mask, &p, &obsvr_port); + kr = vproc_mig_spawn(puc, buf, buf_len, argc, envc, flags, u_mask, bin_pref, binpref_cnt, &p, &obsvr_port); mach_port_deallocate(mach_task_self(), puc); } } Modified: trunk/launchd/src/libvproc_internal.h =================================================================== --- trunk/launchd/src/libvproc_internal.h 2006-12-20 00:48:11 UTC (rev 22993) +++ trunk/launchd/src/libvproc_internal.h 2006-12-21 19:23:38 UTC (rev 22994) @@ -27,6 +27,7 @@ typedef char * _internal_string_t; typedef char * logmsg_t; typedef mach_port_t vproc_mig_t; +typedef integer_t binpref_t[8]; typedef enum { LAST_EXIT_STATUS = 1, GLOBAL_ON_DEMAND, @@ -45,7 +46,6 @@ #define SPAWN_HAS_WDIR 0x0002 #define SPAWN_HAS_UMASK 0x0004 #define SPAWN_WANTS_WAIT4DEBUGGER 0x0008 -#define SPAWN_WANTS_FORCE_PPC 0x0010 kern_return_t _vproc_grab_subset(mach_port_t bp, mach_port_t *reqport, mach_port_t *rcvright, Modified: trunk/launchd/src/protocol_job.defs =================================================================== --- trunk/launchd/src/protocol_job.defs 2006-12-20 00:48:11 UTC (rev 22993) +++ trunk/launchd/src/protocol_job.defs 2006-12-21 19:23:38 UTC (rev 22994) @@ -99,6 +99,8 @@ __envc : uint32_t; __flags : uint64_t; __umask : uint16_t; + __binpref : binpref_t; + __binpref_cnt : uint32_t; out __pid : pid_t; out __obsvr_port : mach_port_make_send_t);

1 0

[22993] tags/launchd-182/
by source_changes＠macosforge.org 20 Dec '06

20 Dec '06

Revision: 22993 http://trac.macosforge.org/projects/launchd/changeset/22993 Author: zarzycki(a)apple.com Date: 2006-12-19 16:48:11 -0800 (Tue, 19 Dec 2006) Log Message: ----------- "Tagging launchd-182 from https://svn.macosforge.org/repository/launchd/trunk" Added Paths: ----------- tags/launchd-182/ Copied: tags/launchd-182 (from rev 22992, trunk)

1 0

[22992] trunk/launchd/src/launchd_core_logic.c
by source_changes＠macosforge.org 19 Dec '06

19 Dec '06

Revision: 22992 http://trac.macosforge.org/projects/launchd/changeset/22992 Author: zarzycki(a)apple.com Date: 2006-12-19 08:38:37 -0800 (Tue, 19 Dec 2006) Log Message: ----------- Use the assumes() macro in more places. Modified Paths: -------------- trunk/launchd/src/launchd_core_logic.c Modified: trunk/launchd/src/launchd_core_logic.c =================================================================== --- trunk/launchd/src/launchd_core_logic.c 2006-12-19 16:35:11 UTC (rev 22991) +++ trunk/launchd/src/launchd_core_logic.c 2006-12-19 16:38:37 UTC (rev 22992) @@ -1977,8 +1977,7 @@ _exit(EXIT_FAILURE); } - if (-1 == setgid(desired_gid)) { - job_log_error(j, LOG_ERR, "setgid(%u)", desired_gid); + if (!job_assumes(j, setgid(desired_gid) != -1)) { _exit(EXIT_FAILURE); } @@ -1988,14 +1987,12 @@ */ if (!j->no_init_groups) { - if (initgroups(loginname, desired_gid) == -1) { - job_log_error(j, LOG_ERR, "initgroups()"); + if (!job_assumes(j, initgroups(loginname, desired_gid) != -1)) { _exit(EXIT_FAILURE); } } - if (-1 == setuid(desired_uid)) { - job_log_error(j, LOG_ERR, "setuid(%u)", desired_uid); + if (!job_assumes(j, setuid(desired_uid) != -1)) { _exit(EXIT_FAILURE); }

1 0

[22991] trunk/launchd/src/launchd_core_logic.c
by source_changes＠macosforge.org 19 Dec '06

19 Dec '06

Revision: 22991 http://trac.macosforge.org/projects/launchd/changeset/22991 Author: zarzycki(a)apple.com Date: 2006-12-19 08:35:11 -0800 (Tue, 19 Dec 2006) Log Message: ----------- <rdar://problem/4866762> getlogin returns "root" instead of correct username Modified Paths: -------------- trunk/launchd/src/launchd_core_logic.c Modified: trunk/launchd/src/launchd_core_logic.c =================================================================== --- trunk/launchd/src/launchd_core_logic.c 2006-12-18 22:30:29 UTC (rev 22990) +++ trunk/launchd/src/launchd_core_logic.c 2006-12-19 16:35:11 UTC (rev 22991) @@ -1973,6 +1973,10 @@ desired_gid = gre->gr_gid; } + if (!job_assumes(j, setlogin(loginname) != -1)) { + _exit(EXIT_FAILURE); + } + if (-1 == setgid(desired_gid)) { job_log_error(j, LOG_ERR, "setgid(%u)", desired_gid); _exit(EXIT_FAILURE);

1 0

[22990] trunk/launchd/src/launchd_core_logic.c
by source_changes＠macosforge.org 18 Dec '06

18 Dec '06

Revision: 22990 http://trac.macosforge.org/projects/launchd/changeset/22990 Author: zarzycki(a)apple.com Date: 2006-12-18 14:30:29 -0800 (Mon, 18 Dec 2006) Log Message: ----------- <rdar://problem/4881060> Pro Tools LE 7.3 fails to launch on Leopard9A322 Modified Paths: -------------- trunk/launchd/src/launchd_core_logic.c Modified: trunk/launchd/src/launchd_core_logic.c =================================================================== --- trunk/launchd/src/launchd_core_logic.c 2006-12-15 22:12:46 UTC (rev 22989) +++ trunk/launchd/src/launchd_core_logic.c 2006-12-18 22:30:29 UTC (rev 22990) @@ -2219,7 +2219,9 @@ job_logv(job_t j, int pri, int err, const char *msg, va_list ap) { char newmsg[10000]; + char *newlabel; int oldmask = 0; + size_t i, o, jlabel_len = strlen(j->label); /* * Hack: If bootstrap_port is set, we must be on the child side of a @@ -2230,10 +2232,21 @@ return _vproc_logv(pri, err, msg, ap); } + newlabel = alloca((jlabel_len + 1) * 2); + + for (i = 0, o = 0; i < jlabel_len; i++, o++) { + if (j->label[i] == '%') { + newlabel[o] = '%'; + o++; + } + newlabel[o] = j->label[i]; + } + newlabel[o] = '\0'; + if (err) { - snprintf(newmsg, sizeof(newmsg), "%s: %s: %s", j->label, msg, strerror(err)); + snprintf(newmsg, sizeof(newmsg), "%s: %s: %s", newlabel, msg, strerror(err)); } else { - snprintf(newmsg, sizeof(newmsg), "%s: %s", j->label, msg); + snprintf(newmsg, sizeof(newmsg), "%s: %s", newlabel, msg); } if (j->debug) {

1 0

[22989] tags/launchd-181/
by source_changes＠macosforge.org 15 Dec '06

15 Dec '06

Revision: 22989 http://trac.macosforge.org/projects/launchd/changeset/22989 Author: zarzycki(a)apple.com Date: 2006-12-15 14:12:46 -0800 (Fri, 15 Dec 2006) Log Message: ----------- "Tagging launchd-181 from https://svn.macosforge.org/repository/launchd/trunk" Added Paths: ----------- tags/launchd-181/ Copied: tags/launchd-181 (from rev 22988, trunk)

1 0

[22988] trunk/launchd/src/launchd.c
by source_changes＠macosforge.org 15 Dec '06

15 Dec '06

Revision: 22988 http://trac.macosforge.org/projects/launchd/changeset/22988 Author: zarzycki(a)apple.com Date: 2006-12-15 14:10:56 -0800 (Fri, 15 Dec 2006) Log Message: ----------- Better self crash reporting in the case of PID 1. Modified Paths: -------------- trunk/launchd/src/launchd.c Modified: trunk/launchd/src/launchd.c =================================================================== --- trunk/launchd/src/launchd.c 2006-12-15 22:10:33 UTC (rev 22987) +++ trunk/launchd/src/launchd.c 2006-12-15 22:10:56 UTC (rev 22988) @@ -105,6 +105,7 @@ static job_t rlcj = NULL; static jmp_buf doom_doom_doom; static void *crash_addr; +static pid_t crash_pid; static const char *launchctl_bootstrap_tool[] = { "/bin/launchctl", /* "bootstrap", */ NULL }; sigset_t blocked_signals = 0; @@ -297,18 +298,24 @@ job_dispatch(fbj, true); } + char *doom_why = "at instruction"; switch (setjmp(doom_doom_doom)) { + case 0: + break; + case SIGBUS: + case SIGSEGV: + doom_why = "trying to read/write"; case SIGILL: case SIGFPE: - syslog(LOG_EMERG, "We crashed at instruction: %p", crash_addr); - abort(); - case SIGBUS: - case SIGSEGV: - syslog(LOG_EMERG, "We crashed trying to read/write: %p", crash_addr); - abort(); + syslog(LOG_EMERG, "We crashed %s: %p (sent by PID %u)", doom_why, crash_addr, crash_pid); default: - abort(); - case 0: + sync(); + sleep(3); + /* the kernel will panic() when PID 1 exits */ + _exit(EXIT_FAILURE); + /* we should never get here */ + reboot(0); + /* or here either */ break; } @@ -338,10 +345,33 @@ launchd_assumes(sigaction(SIGSEGV, &fsa, NULL) != -1); } +#define PID1_CRASH_LOGFILE "/var/log/launchd-pid1.crash" + void fatal_signal_handler(int sig, siginfo_t *si, void *uap) { + char *sample_args[] = { "/usr/bin/sample", "1", "1", "-file", PID1_CRASH_LOGFILE, NULL }; + pid_t sample_p; + int wstatus; + crash_addr = si->si_addr; + crash_pid = si->si_pid; + + unlink(PID1_CRASH_LOGFILE); + + switch ((sample_p = vfork())) { + case 0: + execve(sample_args[0], sample_args, environ); + _exit(EXIT_FAILURE); + break; + default: + waitpid(sample_p, &wstatus, 0); + sync(); + break; + case -1: + break; + } + longjmp(doom_doom_doom, sig); }

1 0