Merge "Reinstate the x86 dynamic linker warning for text relocations."
authorElliott Hughes <enh@google.com>
Mon, 30 Jun 2014 21:55:34 +0000 (21:55 +0000)
committerGerrit Code Review <noreply-gerritcodereview@google.com>
Thu, 26 Jun 2014 18:40:47 +0000 (18:40 +0000)
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));