----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65403/#review196458 -----------------------------------------------------------
3rdparty/stout/include/stout/os/windows/fcntl.hpp Lines 51 (patched) <https://reviews.apache.org/r/65403/#comment276111> s/but all/but/ 3rdparty/stout/include/stout/os/windows/fcntl.hpp Lines 71-72 (patched) <https://reviews.apache.org/r/65403/#comment276112> It may be that we can use the above logic with `SetHandleInformation` on a socket too, but I'd like to do more testing before implementing that. It's not currently required. - Andrew Schwartzmeyer On Jan. 29, 2018, 12:35 p.m., Andrew Schwartzmeyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65403/ > ----------------------------------------------------------- > > (Updated Jan. 29, 2018, 12:35 p.m.) > > > Review request for mesos, Akash Gupta, Jie Yu, and Joseph Wu. > > > Bugs: MESOS-5882 > https://issues.apache.org/jira/browse/MESOS-5882 > > > Repository: mesos > > > Description > ------- > > The intent of `os::cloexec` (that is, the intent of the `CLOEXEC` flags > on Linux) is to disable the inheritance of handles. On Linux, the flag > specifically means to not duplicate the file descriptor on calls to the > `exec` family of functions (which spawn new processes). On Windows, the > logical equivalent is to set `HANDLE_FLAG_INHERIT` to `0`, which means > to not allow the handles (file descriptors) to be inherited (duplicated) > in child processes (exec). > > Note that it is not possible to change the inheritance behavior of an > already-existing socket on Windows, so we log a warning and return > nothing. However, inheritance can be controlled at socket creation, and > so `os::isCloexec` supports sockets too. > > > Diffs > ----- > > 3rdparty/stout/include/stout/os/windows/fcntl.hpp > 5800ec92f85401a80cb813afd880be2e5a24a3af > > > Diff: https://reviews.apache.org/r/65403/diff/1/ > > > Testing > ------- > > > Thanks, > > Andrew Schwartzmeyer > >