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/eigrpd | |
| 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/eigrpd')
| -rw-r--r-- | usr.sbin/eigrpd/eigrpd.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/usr.sbin/eigrpd/eigrpd.c b/usr.sbin/eigrpd/eigrpd.c index fada4d9c8e8..26427d134c2 100644 --- a/usr.sbin/eigrpd/eigrpd.c +++ b/usr.sbin/eigrpd/eigrpd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: eigrpd.c,v 1.26 2018/09/26 14:53:34 mestre Exp $ */ +/* $OpenBSD: eigrpd.c,v 1.27 2019/03/31 03:36:18 yasuoka Exp $ */ /* * Copyright (c) 2015 Renato Westphal <renato@openbsd.org> @@ -25,6 +25,7 @@ #include <err.h> #include <errno.h> +#include <fcntl.h> #include <pwd.h> #include <signal.h> #include <stdio.h> @@ -330,7 +331,10 @@ start_child(enum eigrpd_process p, char *argv0, int fd, int debug, int verbose, return (pid); } - if (dup2(fd, 3) == -1) + if (fd != 3) { + if (dup2(fd, 3) == -1) + fatal("cannot setup imsg fd"); + } else if (fcntl(fd, F_SETFD, 0) == -1) fatal("cannot setup imsg fd"); argv[argc++] = argv0; |