am 260846f3: Merge "Reinstate the x86 dynamic linker warning for text relocations."
authorElliott Hughes <enh@google.com>
Mon, 30 Jun 2014 22:04:33 +0000 (22:04 +0000)
committerAndroid Git Automerger <android-git-automerger@android.com>
Mon, 30 Jun 2014 22:04:33 +0000 (22:04 +0000)
* commit '260846f374f88ba6686bff8ab33c17c487c7a8f7':
  Reinstate the x86 dynamic linker warning for text relocations.

linker/linker.cpp

index b99c59a1830c2c195ff317b240a05b3ab04ab804..45889485f982a895af9195b59181b9dd3b0d6609 100644 (file)
@@ -1929,10 +1929,8 @@ static bool soinfo_link_image(soinfo* si, const android_dlextinfo* extinfo) {
     if (si->has_text_relocations) {
         // Make segments writable to allow text relocations to work properly. We will later call
         // phdr_table_protect_segments() after all of them are applied and all constructors are run.
-#if !defined(__i386__) // The platform itself has too many text relocations on x86.
         DL_WARN("%s has text relocations. This is wasting memory and prevents "
                 "security hardening. Please fix.", si->name);
-#endif
         if (phdr_table_unprotect_segments(si->phdr, si->phnum, si->load_bias) < 0) {
             DL_ERR("can't unprotect loadable segments for \"%s\": %s",
                    si->name, strerror(errno));