[PATCH] D48451: [sanitizers_common] when spawning a subprocess for symbolizers, use posix_spawn instead of fork()

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 22 09:37:02 PDT 2018


eugenis requested changes to this revision.
eugenis added a comment.
This revision now requires changes to proceed.

Tests are crashing with corrupted stack.



================
Comment at: lib/sanitizer_common/sanitizer_linux.cc:800
+#else
+  return internal_syscall(SYSCALL(vfork));
+#endif
----------------
You can't return from a function that calls vfork() in the child process. It will destroy the stack frame in the parent process.



Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D48451





More information about the llvm-commits mailing list