diff options
| author | YASUOKA Masahiko <yasuoka@cvs.openbsd.org> | 2019-03-31 03:36:19 +0000 |
|---|---|---|
| committer | YASUOKA Masahiko <yasuoka@cvs.openbsd.org> | 2019-03-31 03:36:19 +0000 |
| commit | cf399edbc85b744450dfdb02b37efe37d1674cd6 (patch) | |
| tree | c37c8be974fd886794ed8ea97f1302e68ff789e2 /usr.sbin/ldapd | |
| parent | fd28b16511cd31e835d2f1d4bcce7238d07285a6 (diff) | |
Avoid calling dup2(oldd, newd) when oldd == newd. In that case the
descriptor keeps CLOEXEC flag then it will be closed unexpectedly by
exec().
ok tedu florian
Diffstat (limited to 'usr.sbin/ldapd')
| -rw-r--r-- | usr.sbin/ldapd/ldapd.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/usr.sbin/ldapd/ldapd.c b/usr.sbin/ldapd/ldapd.c index 94df93ee4e6..797a36f89f3 100644 --- a/usr.sbin/ldapd/ldapd.c +++ b/usr.sbin/ldapd/ldapd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ldapd.c,v 1.24 2018/05/15 11:19:21 reyk Exp $ */ +/* $OpenBSD: ldapd.c,v 1.25 2019/03/31 03:36:18 yasuoka Exp $ */ /* * Copyright (c) 2009, 2010 Martin Hedenfalk <martin@bzero.se> @@ -429,7 +429,10 @@ start_child(enum ldapd_process p, char *argv0, int fd, int debug, return (pid); } - if (dup2(fd, PROC_PARENT_SOCK_FILENO) == -1) + if (fd != PROC_PARENT_SOCK_FILENO) { + if (dup2(fd, PROC_PARENT_SOCK_FILENO) == -1) + fatal("cannot setup imsg fd"); + } else if (fcntl(fd, F_SETFD, 0) == -1) fatal("cannot setup imsg fd"); argv[argc++] = argv0; |