diff --git a/src/node.cc b/src/node.cc index 2891c18bb9aa9a..7379fdab7c1eb0 100644 --- a/src/node.cc +++ b/src/node.cc @@ -608,11 +608,32 @@ static void PlatformInit(ProcessInitializationFlags::Flags flags) { for (auto& s : stdio) { const int fd = &s - stdio; if (fstat(fd, &s.stat) == 0) continue; + // Anything but EBADF means something is seriously wrong. We don't // have to special-case EINTR, fstat() is not interruptible. if (errno != EBADF) ABORT(); - if (fd != open("/dev/null", O_RDWR)) ABORT(); - if (fstat(fd, &s.stat) != 0) ABORT(); + + // If EBADF (file descriptor doesn't exist), open /dev/null and duplicate + // its file descriptor to the invalid file descriptor. Make sure *that* + // file descriptor is valid. POSIX doesn't guarantee the next file + // descriptor open(2) gives us is the lowest available number anymore in + // POSIX.1-2017, which is why dup2(2) is needed. + int null_fd; + + do { + null_fd = open("/dev/null", O_RDWR); + } while (null_fd < 0 && errno == EINTR); + + if (null_fd != fd) { + int err; + + do { + err = dup2(null_fd, fd); + } while (err < 0 && errno == EINTR); + CHECK_EQ(err, 0); + } + + if (fstat(fd, &s.stat) < 0) ABORT(); } }