commit ec79de0907aeecfe4cb94cc1fb0510ba148eb8c5
author Elliott Hughes <enh@google.com> Fri Jan 17 07:20:37 2025 -0800
committer Elliott Hughes <enh@google.com> Fri Jan 17 07:28:45 2025 -0800
tree 037d49ab65a7b76d3ffd1f35b4ce85bf67a4ff8d
parent a7ac0efcca42cb31949139ade5f54d8e486c1fe6

Skip rather than reject invalid ELF notes for existing apps.

Bug: http://b/390328213
Change-Id: I322653266c40ef3084d6cc9e8fdb2c5218c8cc65

linker/linker_phdr.cpp

diff --git a/linker/linker_phdr.cpp b/linker/linker_phdr.cpp
index 086c867..2eadbf9 100644
--- a/linker/linker_phdr.cpp
+++ b/linker/linker_phdr.cpp
@@ -772,8 +772,14 @@
         __builtin_add_overflow(note_end_off, phdr->p_filesz, &note_end_off) ||
         phdr->p_filesz != phdr->p_memsz ||
         note_end_off > file_size_) {
-      DL_ERR("\"%s\": NT_ANDROID_TYPE_PAD_SEGMENT note runs off end of file",
-             name_.c_str());
+
+      if (get_application_target_sdk_version() < 37) {
+        // Some in-market apps have invalid ELF notes (http://b/390328213),
+        // so ignore them until/unless they bump their target sdk version.
+        continue;
+      }
+
+      DL_ERR_AND_LOG("\"%s\": NT_ANDROID_TYPE_PAD_SEGMENT note runs off end of file", name_.c_str());
       return false;
     }