commit 725457a9e46a773f0fd5e18c9cf0f00b2347dcaf from: Omar Polo date: Sun Jun 11 09:26:16 2023 UTC move setproctitle/privsep_process earlier We don't always do privilege dropping (as we may start as unprivileged user), so set these two beforehand so when we skip privdrop we don't miss to set privsep_process and set the process' title. commit - 4ad573d0d5675212b0b5719a0a5c1de22974dd0e commit + 725457a9e46a773f0fd5e18c9cf0f00b2347dcaf blob - 3060e65789708c425f400e0c13b5fc4a8609f0b3 blob + 79eaa328ceace4ed2d776483e1c92048cf2ad2ce --- proc.c +++ proc.c @@ -548,7 +548,10 @@ proc_run(struct privsep *ps, struct privsep_proc *p, const char *root; log_procinit(p->p_title); + setproctitle("%s", p->p_title); + privsep_process = p->p_id; + if (ps->ps_pw == NULL) goto init; @@ -572,10 +575,6 @@ proc_run(struct privsep *ps, struct privsep_proc *p, if (chdir("/") == -1) fatal("%s: chdir(\"/\")", __func__); - privsep_process = p->p_id; - - setproctitle("%s", p->p_title); - if (setgroups(1, &pw->pw_gid) || setresgid(pw->pw_gid, pw->pw_gid, pw->pw_gid) || setresuid(pw->pw_uid, pw->pw_uid, pw->pw_uid))