[PATCH] D14869: [sanitizer] Implement internal_fork and internal_forkpty for OS X

Dmitry Vyukov via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 20 05:46:03 PST 2015


dvyukov accepted this revision.
dvyukov added a comment.
This revision is now accepted and ready to land.

LGTM with a nit


================
Comment at: lib/sanitizer_common/sanitizer_mac.cc:162
@@ +161,3 @@
+  int pid = __fork();
+  if (pid == -1) return -1;
+  if (pid == 0) {
----------------
add:

  close(master);
  close(slave);


http://reviews.llvm.org/D14869





More information about the llvm-commits mailing list