[PATCH] D75303: Update Sanitizer tests for Android R.

Mitch Phillips via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 27 13:47:18 PST 2020


hctim marked an inline comment as done.
hctim added inline comments.


================
Comment at: compiler-rt/test/asan/TestCases/Posix/no-fd.cpp:7
 
+// This test closes the 0, 1, and 2 file descriptors before an exec() and relies
+// on them remaining closed across an execve(). This is not the case on newer
----------------
"As a general principle, no portable program, whether privileged or not, can assume that these three file descriptors will remain closed across an execve()."
 - http://man7.org/linux/man-pages/man2/execve.2.html



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75303/new/

https://reviews.llvm.org/D75303





More information about the llvm-commits mailing list