From: florian Date: Sun, 19 Aug 2018 12:31:41 +0000 (+0000) Subject: Call daemon with 0 as first argument so that it changes the cwd to /. X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=83926339880de9648b94ba4943d1dbf509b83d99;p=openbsd Call daemon with 0 as first argument so that it changes the cwd to /. It is not a problem in slowcgi since it calls chroot(2) and then chdir(2) shortly afterwards but hopefully prevents copying code into daemons where it does matter. Problem first observed by dlg in ntpd(8) which keeps sitting in the directory from where it was started which might for example prevent an unmount. Discussed with deraadt@ --- diff --git a/usr.sbin/slowcgi/slowcgi.c b/usr.sbin/slowcgi/slowcgi.c index 8d8c970d861..696af7228f2 100644 --- a/usr.sbin/slowcgi/slowcgi.c +++ b/usr.sbin/slowcgi/slowcgi.c @@ -1,4 +1,4 @@ -/* $OpenBSD: slowcgi.c,v 1.53 2018/08/13 16:54:50 florian Exp $ */ +/* $OpenBSD: slowcgi.c,v 1.54 2018/08/19 12:31:41 florian Exp $ */ /* * Copyright (c) 2013 David Gwynne * Copyright (c) 2013 Florian Obser @@ -323,7 +323,7 @@ main(int argc, char *argv[]) if (geteuid() != 0) errx(1, "need root privileges"); - if (!debug && daemon(1, 0) == -1) + if (!debug && daemon(0, 0) == -1) err(1, "daemon"); if (!debug) {