* elf.c (assign_file_positions_for_non_load_sections): Consolidate
authorAlan Modra <amodra@gmail.com>
Tue, 11 Nov 2008 04:26:13 +0000 (04:26 +0000)
committerAlan Modra <amodra@gmail.com>
Tue, 11 Nov 2008 04:26:13 +0000 (04:26 +0000)
PT_GNU_RELRO handling.

bfd/ChangeLog
bfd/elf.c

index 86aa2765530aa95bd1337cbbb792e89ca39eca94..c539cf7f5ee4fcd50930a457937d897863f13201 100644 (file)
@@ -1,3 +1,8 @@
+2008-11-11  Alan Modra  <amodra@bigpond.net.au>
+
+       * elf.c (assign_file_positions_for_non_load_sections): Consolidate
+       PT_GNU_RELRO handling.
+
 2008-11-11  Alan Modra  <amodra@bigpond.net.au>
 
        PR 7012
index 2f591f52d5e41aa4e08de5ac419e6e0dd9b3a613..054fd2a410539e29e3c39017f7347f1848ab464d 100644 (file)
--- a/bfd/elf.c
+++ b/bfd/elf.c
@@ -4633,71 +4633,57 @@ assign_file_positions_for_non_load_sections (bfd *abfd,
     {
       if (p->p_type == PT_GNU_RELRO)
        {
+         const Elf_Internal_Phdr *lp;
+
          BFD_ASSERT (!m->includes_filehdr && !m->includes_phdrs);
 
          if (link_info != NULL)
            {
              /* During linking the range of the RELRO segment is passed
                 in link_info.  */
-             Elf_Internal_Phdr *lp;
-
              for (lp = phdrs; lp < phdrs + count; ++lp)
                {
                  if (lp->p_type == PT_LOAD
-                     && lp->p_vaddr <= link_info->relro_end
                      && lp->p_vaddr >= link_info->relro_start
-                     && (lp->p_vaddr + lp->p_filesz
-                         >= link_info->relro_end))
+                     && lp->p_vaddr < link_info->relro_end
+                     && lp->p_vaddr + lp->p_filesz >= link_info->relro_end)
                    break;
                }
-
-             if (lp < phdrs + count
-                 && link_info->relro_end > lp->p_vaddr)
-               {
-                 p->p_vaddr = lp->p_vaddr;
-                 p->p_paddr = lp->p_paddr;
-                 p->p_offset = lp->p_offset;
-                 p->p_filesz = link_info->relro_end - lp->p_vaddr;
-                 p->p_memsz = p->p_filesz;
-                 p->p_align = 1;
-                 p->p_flags = (lp->p_flags & ~PF_W);
-               }
-             else
-               {
-                 memset (p, 0, sizeof *p);
-                 p->p_type = PT_NULL;
-               }
            }
          else
            {
              /* Otherwise we are copying an executable or shared
-                library. But we need to use the same linker logic.  */
-             Elf_Internal_Phdr *lp;
-
+                library, but we need to use the same linker logic.  */
              for (lp = phdrs; lp < phdrs + count; ++lp)
                {
                  if (lp->p_type == PT_LOAD
                      && lp->p_paddr == p->p_paddr)
                    break;
                }
-         
-             if (lp < phdrs + count)
-               {
-                 /* We should use p_size if it is valid since it
-                    may contain the first few bytes of the next
-                    SEC_ALLOC section.  */
-                 if (m->p_size_valid)
-                   p->p_filesz = m->p_size;
-                 else
-                   abort ();
-                 p->p_vaddr = lp->p_vaddr;
-                 p->p_offset = lp->p_offset;
-                 p->p_memsz = p->p_filesz;
-                 p->p_align = 1;
-               }
+           }
+
+         if (lp < phdrs + count)
+           {
+             p->p_vaddr = lp->p_vaddr;
+             p->p_paddr = lp->p_paddr;
+             p->p_offset = lp->p_offset;
+             if (link_info != NULL)
+               p->p_filesz = link_info->relro_end - lp->p_vaddr;
+             else if (m->p_size_valid)
+               p->p_filesz = m->p_size;
              else
                abort ();
+             p->p_memsz = p->p_filesz;
+             p->p_align = 1;
+             p->p_flags = (lp->p_flags & ~PF_W);
            }
+         else if (link_info != NULL)
+           {
+             memset (p, 0, sizeof *p);
+             p->p_type = PT_NULL;
+           }
+         else
+           abort ();
        }
       else if (m->count != 0)
        {