[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