This is an archive of the discontinued LLVM Phabricator instance.

Update Sanitizer tests for Android R.
ClosedPublic

Authored by hctim on Feb 27 2020, 1:42 PM.

Details

Summary

Sanitizer tests don't entirely pass on an R device. Fix up all the
incompatibilities with the new system.

Diff Detail

Event Timeline

hctim created this revision.Feb 27 2020, 1:42 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 27 2020, 1:42 PM
Herald added subscribers: llvm-commits, Restricted Project. · View Herald Transcript
hctim marked an inline comment as done.Feb 27 2020, 1:44 PM
hctim added inline comments.
compiler-rt/test/asan/TestCases/Posix/no-fd.cpp
7

"As a general principle, no portable program, whether privileged or not, can assume that these three file descriptors will remain closed across an execve()."

eugenis accepted this revision.Feb 27 2020, 1:52 PM

LGTM

This revision is now accepted and ready to land.Feb 27 2020, 1:52 PM
This revision was automatically updated to reflect the committed changes.