diff options
| author | Elliott Hughes <enh@google.com> | 2016-11-16 19:04:17 +0000 |
|---|---|---|
| committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2016-11-16 19:04:18 +0000 |
| commit | 8945d5e43a25737f3f888a90742cc918b1124a05 (patch) | |
| tree | 6757a77ed8efe27ee2f5e9dbf0e53144b1a26568 /linker | |
| parent | bd15515cfc0219963123bcf982782100ad0f6085 (diff) | |
| parent | 4cc5a60f9113d7cb5bae01334df06e6ccd0018e5 (diff) | |
| download | android_bionic-8945d5e43a25737f3f888a90742cc918b1124a05.tar.gz android_bionic-8945d5e43a25737f3f888a90742cc918b1124a05.tar.bz2 android_bionic-8945d5e43a25737f3f888a90742cc918b1124a05.zip | |
Merge "Downgrade W+E load segments to a warning unless targeting O."
Diffstat (limited to 'linker')
| -rw-r--r-- | linker/linker_phdr.cpp | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/linker/linker_phdr.cpp b/linker/linker_phdr.cpp index bced72296..5b0ee491a 100644 --- a/linker/linker_phdr.cpp +++ b/linker/linker_phdr.cpp @@ -36,6 +36,7 @@ #include <unistd.h> #include "linker.h" +#include "linker_dlwarning.h" #include "linker_globals.h" #include "linker_debug.h" #include "linker_utils.h" @@ -605,10 +606,14 @@ bool ElfReader::LoadSegments() { if (file_length != 0) { int prot = PFLAGS_TO_PROT(phdr->p_flags); - // W + E PT_LOAD segments are not allowed. if ((prot & (PROT_EXEC | PROT_WRITE)) == (PROT_EXEC | PROT_WRITE)) { - DL_ERR_AND_LOG("\"%s\": W + E load segments are not allowed", name_.c_str()); - return false; + // W + E PT_LOAD segments are not allowed in O. + if (get_application_target_sdk_version() > 25) { + DL_ERR_AND_LOG("\"%s\": W + E load segments are not allowed", name_.c_str()); + return false; + } + DL_WARN("\"%s\": W + E load segments are not allowed", name_.c_str()); + add_dlwarning(name_.c_str(), "W+E load segments"); } void* seg_addr = mmap64(reinterpret_cast<void*>(seg_page_start), |