diff options
| author | H.J. Lu <hjl.tools@gmail.com> | 2007-10-11 05:03:07 +0000 |
|---|---|---|
| committer | H.J. Lu <hjl.tools@gmail.com> | 2007-10-11 05:03:07 +0000 |
| commit | f210dcff90d54b8ae46967a3a4b07757cacd959f (patch) | |
| tree | 5b13f3c4584eeca53cf09f5ec885177ccdcb2d5f /bfd | |
| parent | 21dec5f51af65996b82580eebd9a8914286347a1 (diff) | |
| download | gdb-f210dcff90d54b8ae46967a3a4b07757cacd959f.zip gdb-f210dcff90d54b8ae46967a3a4b07757cacd959f.tar.gz gdb-f210dcff90d54b8ae46967a3a4b07757cacd959f.tar.bz2 | |
2007-10-10 H.J. Lu <hongjiu.lu@intel.com>
* elf.c (get_program_header_size): Always add a PT_GNU_RELRO
segment for -z relro.
(_bfd_elf_map_sections_to_segments): Make a PT_GNU_RELRO
segment only when needed.
Diffstat (limited to 'bfd')
| -rw-r--r-- | bfd/ChangeLog | 7 | ||||
| -rw-r--r-- | bfd/elf.c | 54 |
2 files changed, 42 insertions, 19 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 9117508..9e8ffad 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,10 @@ +2007-10-10 H.J. Lu <hongjiu.lu@intel.com> + + * elf.c (get_program_header_size): Always add a PT_GNU_RELRO + segment for -z relro. + (_bfd_elf_map_sections_to_segments): Make a PT_GNU_RELRO + segment only when needed. + 2007-10-05 Bob Wilson <bob.wilson@acm.org> * elf32-xtensa.c (relax_section): Call pin_internal_relocs when @@ -3358,13 +3358,12 @@ get_program_header_size (bfd *abfd, struct bfd_link_info *info) { /* We need a PT_DYNAMIC segment. */ ++segs; + } - if (info->relro) - { - /* We need a PT_GNU_RELRO segment only when there is a - PT_DYNAMIC segment. */ - ++segs; - } + if (info->relro) + { + /* We need a PT_GNU_RELRO segment. */ + ++segs; } if (elf_tdata (abfd)->eh_frame_hdr) @@ -3889,21 +3888,38 @@ _bfd_elf_map_sections_to_segments (bfd *abfd, struct bfd_link_info *info) pm = &m->next; } - if (dynsec != NULL && info->relro) + if (info->relro) { - /* We make a PT_GNU_RELRO segment only when there is a - PT_DYNAMIC segment. */ - amt = sizeof (struct elf_segment_map); - m = bfd_zalloc (abfd, amt); - if (m == NULL) - goto error_return; - m->next = NULL; - m->p_type = PT_GNU_RELRO; - m->p_flags = PF_R; - m->p_flags_valid = 1; + for (m = mfirst; m != NULL; m = m->next) + { + if (m->p_type == PT_LOAD) + { + asection *last = m->sections[m->count - 1]; + bfd_vma vaddr = m->sections[0]->vma; + bfd_vma filesz = last->vma - vaddr + last->size; - *pm = m; - pm = &m->next; + if (vaddr < info->relro_end + && vaddr >= info->relro_start + && (vaddr + filesz) >= info->relro_end) + break; + } + } + + /* Make a PT_GNU_RELRO segment only when it isn't empty. */ + if (m != NULL) + { + amt = sizeof (struct elf_segment_map); + m = bfd_zalloc (abfd, amt); + if (m == NULL) + goto error_return; + m->next = NULL; + m->p_type = PT_GNU_RELRO; + m->p_flags = PF_R; + m->p_flags_valid = 1; + + *pm = m; + pm = &m->next; + } } free (sections); |