94084c
commit 5f36e5c70107ecb59281ef57f9f1c0e37ec3076d
94084c
Author: H.J. Lu <hjl.tools@gmail.com>
94084c
Date:   Thu Sep 23 09:06:49 2021 -0700
94084c
94084c
    ld.so: Initialize bootstrap_map.l_ld_readonly [BZ #28340]
94084c
    
94084c
    1. Define DL_RO_DYN_SECTION to initalize bootstrap_map.l_ld_readonly
94084c
    before calling elf_get_dynamic_info to get dynamic info in bootstrap_map,
94084c
    2. Define a single
94084c
    
94084c
    static inline bool
94084c
    dl_relocate_ld (const struct link_map *l)
94084c
    {
94084c
      /* Don't relocate dynamic section if it is readonly  */
94084c
      return !(l->l_ld_readonly || DL_RO_DYN_SECTION);
94084c
    }
94084c
    
94084c
    This updates BZ #28340 fix.
94084c
    
94084c
    (cherry picked from commit 2ec99d8c42b2ff1a1231e4df462a0910a9b7fdef)
94084c
94084c
diff --git a/elf/rtld.c b/elf/rtld.c
94084c
index 405166d62b34847c..d83ac1bdc40a6081 100644
94084c
--- a/elf/rtld.c
94084c
+++ b/elf/rtld.c
94084c
@@ -548,6 +548,7 @@ _dl_start (void *arg)
94084c
 
94084c
   /* Read our own dynamic section and fill in the info array.  */
94084c
   bootstrap_map.l_ld = (void *) bootstrap_map.l_addr + elf_machine_dynamic ();
94084c
+  bootstrap_map.l_ld_readonly = DL_RO_DYN_SECTION;
94084c
   elf_get_dynamic_info (&bootstrap_map);
94084c
 
94084c
 #if NO_TLS_OFFSET != 0
94084c
diff --git a/sysdeps/generic/dl-relocate-ld.h b/sysdeps/generic/dl-relocate-ld.h
94084c
index 5fae206db9941e97..cfb86c2d6a2394c1 100644
94084c
--- a/sysdeps/generic/dl-relocate-ld.h
94084c
+++ b/sysdeps/generic/dl-relocate-ld.h
94084c
@@ -19,14 +19,7 @@
94084c
 #ifndef _DL_RELOCATE_LD_H
94084c
 #define _DL_RELOCATE_LD_H
94084c
 
94084c
-/* Return true if dynamic section in the shared library L should be
94084c
-   relocated.  */
94084c
-
94084c
-static inline bool
94084c
-dl_relocate_ld (const struct link_map *l)
94084c
-{
94084c
-  /* Don't relocate dynamic section if it is readonly  */
94084c
-  return !l->l_ld_readonly;
94084c
-}
94084c
+/* The dynamic section is writable.  */
94084c
+#define DL_RO_DYN_SECTION 0
94084c
 
94084c
 #endif /* _DL_RELOCATE_LD_H */
94084c
diff --git a/sysdeps/generic/ldsodefs.h b/sysdeps/generic/ldsodefs.h
94084c
index ed10953f34b96c49..fcbbf6974827cdf1 100644
94084c
--- a/sysdeps/generic/ldsodefs.h
94084c
+++ b/sysdeps/generic/ldsodefs.h
94084c
@@ -69,6 +69,16 @@ __BEGIN_DECLS
94084c
    `ElfW(TYPE)' is used in place of `Elf32_TYPE' or `Elf64_TYPE'.  */
94084c
 #define ELFW(type)	_ElfW (ELF, __ELF_NATIVE_CLASS, type)
94084c
 
94084c
+/* Return true if dynamic section in the shared library L should be
94084c
+   relocated.  */
94084c
+
94084c
+static inline bool
94084c
+dl_relocate_ld (const struct link_map *l)
94084c
+{
94084c
+  /* Don't relocate dynamic section if it is readonly  */
94084c
+  return !(l->l_ld_readonly || DL_RO_DYN_SECTION);
94084c
+}
94084c
+
94084c
 /* All references to the value of l_info[DT_PLTGOT],
94084c
   l_info[DT_STRTAB], l_info[DT_SYMTAB], l_info[DT_RELA],
94084c
   l_info[DT_REL], l_info[DT_JMPREL], and l_info[VERSYMIDX (DT_VERSYM)]
94084c
diff --git a/sysdeps/mips/dl-relocate-ld.h b/sysdeps/mips/dl-relocate-ld.h
94084c
index 0c18d9a567cad54f..376ad75dd1695fe3 100644
94084c
--- a/sysdeps/mips/dl-relocate-ld.h
94084c
+++ b/sysdeps/mips/dl-relocate-ld.h
94084c
@@ -19,14 +19,7 @@
94084c
 #ifndef _DL_RELOCATE_LD_H
94084c
 #define _DL_RELOCATE_LD_H
94084c
 
94084c
-/* Return true if dynamic section in the shared library L should be
94084c
-   relocated.  */
94084c
-
94084c
-static inline bool
94084c
-dl_relocate_ld (const struct link_map *l)
94084c
-{
94084c
-  /* Never relocate dynamic section.  */
94084c
-  return false;
94084c
-}
94084c
+/* The dynamic section is readonly.  */
94084c
+#define DL_RO_DYN_SECTION 1
94084c
 
94084c
 #endif /* _DL_RELOCATE_LD_H */
94084c
diff --git a/sysdeps/riscv/dl-relocate-ld.h b/sysdeps/riscv/dl-relocate-ld.h
94084c
index 10327454b17a7097..2ab2b8ac6c09af82 100644
94084c
--- a/sysdeps/riscv/dl-relocate-ld.h
94084c
+++ b/sysdeps/riscv/dl-relocate-ld.h
94084c
@@ -19,14 +19,7 @@
94084c
 #ifndef _DL_RELOCATE_LD_H
94084c
 #define _DL_RELOCATE_LD_H
94084c
 
94084c
-/* Return true if dynamic section in the shared library L should be
94084c
-   relocated.  */
94084c
-
94084c
-static inline bool
94084c
-dl_relocate_ld (const struct link_map *l)
94084c
-{
94084c
-  /* Never relocate dynamic section for ABI compatibility.  */
94084c
-  return false;
94084c
-}
94084c
+/* The dynamic section is readonly for ABI compatibility.  */
94084c
+#define DL_RO_DYN_SECTION 1
94084c
 
94084c
 #endif /* _DL_RELOCATE_LD_H */