Changeset 23024

Timestamp:

01/26/07 15:45:13 (23 months ago)

Author:

zarzycki@…

Message:

PIDs shouldn't be running around without user records to back the UIDs assigned to the process.

Files:

• trunk/launchd/src/launchd_core_logic.c (modified) (3 diffs)

trunk/launchd/src/launchd_core_logic.c

@@ -405 +405 @@
 {
         if (j->p) {
-                kill(j->p, SIGTERM);
+                job_assumes(j, kill(j->p, SIGTERM) != -1);
         }
 }
@@ -1976 +1976 @@
                 uid_t i_uid = kp[i].kp_eproc.e_pcred.p_ruid;
                 uid_t i_svuid = kp[i].kp_eproc.e_pcred.p_svuid;
+                pid_t i_pid = kp[i].kp_proc.p_pid;
 
                 if (i_euid != u && i_uid != u && i_svuid != u) {
@@ -1982 +1983 @@
 
                 job_log(j, LOG_ERR, "PID %u \"%s\" has no account to back it! Real/effective/saved UIDs: %u/%u/%u",
-                                kp[i].kp_proc.p_pid, kp[i].kp_proc.p_comm, i_uid, i_euid, i_svuid);
+                                i_pid, kp[i].kp_proc.p_comm, i_uid, i_euid, i_svuid);
+
+                /* Ask the accountless process to exit. */
+                job_assumes(j, kill(i_pid, SIGTERM) != -1);
         }