diff options
| author | Kane York <kanepyork@gmail.com> | 2020-07-24T02·10-0700 |
|---|---|---|
| committer | kanepyork <rikingcoding@gmail.com> | 2020-07-24T22·16+0000 |
| commit | bd770907034609dcdb927c3a75b6a98eff4f23e7 (patch) | |
| tree | ae8ac85322ebf8bbf67ffd38ce91db50f7359b6e /third_party/nix/src/nix-daemon | |
| parent | 388b5f1abe8947978592d9778a5669e634b6e552 (diff) | |
fix(3p/nix): do not call vfork r/1457
The use of vfork() in Nix is entirely illegal. Quote: If the process created by vfork() returns from the function in which vfork() was called, or calls any other function before successfully calling _exit() or one of the exec*() family of functions, the behavior is undefined. -- Linux man-pages, release 5.05 Add a TODO to use the higher-performance variants of clone() on Linux when it is available. Change-Id: I42370e1568ad6e2d00d70d0b66c8aded8f1288bb Reviewed-on: https://cl.tvl.fyi/c/depot/+/1418 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in> Reviewed-by: Alyssa Ross <hi@alyssa.is>
Diffstat (limited to 'third_party/nix/src/nix-daemon')
| -rw-r--r-- | third_party/nix/src/nix-daemon/nix-daemon.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/third_party/nix/src/nix-daemon/nix-daemon.cc b/third_party/nix/src/nix-daemon/nix-daemon.cc index 1f49788ae573..2570179d42f1 100644 --- a/third_party/nix/src/nix-daemon/nix-daemon.cc +++ b/third_party/nix/src/nix-daemon/nix-daemon.cc @@ -1051,7 +1051,6 @@ static void daemonLoop(char** argv) { options.errorPrefix = "unexpected Nix daemon error: "; options.dieWithParent = false; options.runExitHandlers = true; - options.allowVfork = false; startProcess( [&]() { fdSocket = -1; |