diff options
author | hselasky <hselasky@FreeBSD.org> | 2015-10-13 08:21:15 +0000 |
---|---|---|
committer | hselasky <hselasky@FreeBSD.org> | 2015-10-13 08:21:15 +0000 |
commit | 8096b85f202b18e1be6a5dd5251a10fcdb255a78 (patch) | |
tree | 1b007a7aaac7c40e942031cbd037d0d85d290550 /usr.bin | |
parent | 6cdda4a6562385b1c65f22415b4249090fc3118a (diff) | |
download | FreeBSD-src-8096b85f202b18e1be6a5dd5251a10fcdb255a78.zip FreeBSD-src-8096b85f202b18e1be6a5dd5251a10fcdb255a78.tar.gz |
MFC r288335:
Store PID after becoming a daemon() and not before to ensure the
correct PID gets written to the PID file.
PR: 203252
Diffstat (limited to 'usr.bin')
-rw-r--r-- | usr.bin/usbhidaction/usbhidaction.c | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/usr.bin/usbhidaction/usbhidaction.c b/usr.bin/usbhidaction/usbhidaction.c index 876dbca..49cf0af 100644 --- a/usr.bin/usbhidaction/usbhidaction.c +++ b/usr.bin/usbhidaction/usbhidaction.c @@ -166,17 +166,15 @@ main(int argc, char **argv) if (demon) { fp = open(pidfile, O_WRONLY|O_CREAT, S_IRUSR|S_IRGRP|S_IROTH); - if (fp >= 0) { - sz1 = snprintf(buf, sizeof buf, "%ld\n", - (long)getpid()); - if (sz1 > sizeof buf) - sz1 = sizeof buf; - write(fp, buf, sz1); - close(fp); - } else + if (fp < 0) err(1, "%s", pidfile); if (daemon(0, 0) < 0) err(1, "daemon()"); + snprintf(buf, sizeof(buf), "%ld\n", (long)getpid()); + sz1 = strlen(buf); + if (write(fp, buf, sz1) < 0) + err(1, "%s", pidfile); + close(fp); isdemon = 1; } |