From 0920dee7bad8a96ae1866b9cd451103d8897abf5 Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Thu, 22 Mar 2007 15:50:38 +0000 Subject: [PATCH] bfd/ 2007-03-22 H.J. Lu PR ld/4007 * elf.c (assign_file_positions_for_load_sections): Check if all sections are in the segment. ld/testsuite/ 2007-03-22 H.J. Lu PR ld/4007 * ld-elf/note-1.d: New file. * ld-elf/note-1.s: Likewise. * ld-elf/note-1.t: Likewise. * ld-i386/alloc.d: Likewise. * ld-i386/alloc.s: Likewise. * ld-i386/alloc.t: Likewise. * ld-i386/i386.exp: Run "alloc". --- bfd/ChangeLog | 6 ++++++ bfd/elf.c | 29 ++++++++++++++++++++++++++--- ld/testsuite/ChangeLog | 12 ++++++++++++ ld/testsuite/ld-elf/note-1.d | 8 ++++++++ ld/testsuite/ld-elf/note-1.s | 6 ++++++ ld/testsuite/ld-elf/note-1.t | 14 ++++++++++++++ ld/testsuite/ld-i386/alloc.d | 4 ++++ ld/testsuite/ld-i386/alloc.s | 6 ++++++ ld/testsuite/ld-i386/alloc.t | 13 +++++++++++++ ld/testsuite/ld-i386/i386.exp | 1 + 10 files changed, 96 insertions(+), 3 deletions(-) create mode 100644 ld/testsuite/ld-elf/note-1.d create mode 100644 ld/testsuite/ld-elf/note-1.s create mode 100644 ld/testsuite/ld-elf/note-1.t create mode 100644 ld/testsuite/ld-i386/alloc.d create mode 100644 ld/testsuite/ld-i386/alloc.s create mode 100644 ld/testsuite/ld-i386/alloc.t diff --git a/bfd/ChangeLog b/bfd/ChangeLog index e33adfb..7c6bc2e 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,9 @@ +2007-03-22 H.J. Lu + + PR ld/4007 + * elf.c (assign_file_positions_for_load_sections): Check if + all sections are in the segment. + 2003-03-20 H.J. Lu PR ld/4208 diff --git a/bfd/elf.c b/bfd/elf.c index 0a6eaa0..3777838 100644 --- a/bfd/elf.c +++ b/bfd/elf.c @@ -4246,7 +4246,7 @@ assign_file_positions_for_load_sections (bfd *abfd, file_ptr off, voff; bfd_size_type maxpagesize; unsigned int alloc; - unsigned int i; + unsigned int i, j; if (link_info == NULL && !elf_modify_segment_map (abfd, link_info)) @@ -4284,9 +4284,9 @@ assign_file_positions_for_load_sections (bfd *abfd, off = bed->s->sizeof_ehdr; off += alloc * bed->s->sizeof_phdr; - for (m = elf_tdata (abfd)->segment_map, p = phdrs; + for (m = elf_tdata (abfd)->segment_map, p = phdrs, j = 0; m != NULL; - m = m->next, p++) + m = m->next, p++, j++) { asection **secpp; @@ -4604,6 +4604,29 @@ assign_file_positions_for_load_sections (bfd *abfd, p->p_flags |= PF_W; } } + + /* Check if all sections are in the segment. Skip PT_GNU_RELRO + and PT_NOTE segments since they will be processed by + assign_file_positions_for_non_load_sections later. */ + if (p->p_type != PT_GNU_RELRO + && p->p_type != PT_NOTE) + for (i = 0, secpp = m->sections; i < m->count; i++, secpp++) + { + Elf_Internal_Shdr *this_hdr; + asection *sec; + + sec = *secpp; + this_hdr = &(elf_section_data(sec)->this_hdr); + if (this_hdr->sh_size != 0 + && !ELF_IS_SECTION_IN_SEGMENT_FILE (this_hdr, p)) + { + (*_bfd_error_handler) + (_("%B: section `%A' can't be allocated in segment %d"), + abfd, sec, j); + bfd_set_error (bfd_error_bad_value); + return FALSE; + } + } } elf_tdata (abfd)->next_file_pos = off; diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog index 20f932a..07f2895 100644 --- a/ld/testsuite/ChangeLog +++ b/ld/testsuite/ChangeLog @@ -1,3 +1,15 @@ +2007-03-22 H.J. Lu + + PR ld/4007 + * ld-elf/note-1.d: New file. + * ld-elf/note-1.s: Likewise. + * ld-elf/note-1.t: Likewise. + * ld-i386/alloc.d: Likewise. + * ld-i386/alloc.s: Likewise. + * ld-i386/alloc.t: Likewise. + + * ld-i386/i386.exp: Run "alloc". + 2007-03-20 Paul Brook * ld-arm/arm-elf.exp (ld-arm/arm-elf.exp): Add arm-pic-veneer. diff --git a/ld/testsuite/ld-elf/note-1.d b/ld/testsuite/ld-elf/note-1.d new file mode 100644 index 0000000..a5fc40f --- /dev/null +++ b/ld/testsuite/ld-elf/note-1.d @@ -0,0 +1,8 @@ +#ld: -Tnote-1.t +#readelf: -l --wide + +#... + Section to Segment mapping: + Segment Sections... + 00 .foo + 01 .note diff --git a/ld/testsuite/ld-elf/note-1.s b/ld/testsuite/ld-elf/note-1.s new file mode 100644 index 0000000..82c9513 --- /dev/null +++ b/ld/testsuite/ld-elf/note-1.s @@ -0,0 +1,6 @@ + .globl _entry + .section .foo,"awx",%progbits +_entry: + .byte 0 + .section .note,"",%note + .byte 0 diff --git a/ld/testsuite/ld-elf/note-1.t b/ld/testsuite/ld-elf/note-1.t new file mode 100644 index 0000000..031fe82 --- /dev/null +++ b/ld/testsuite/ld-elf/note-1.t @@ -0,0 +1,14 @@ +ENTRY(_entry) +PHDRS +{ + data PT_LOAD; + note PT_NOTE; +} +SECTIONS +{ + . = 0x1000000; + .foo : { *(.foo) } :data + . = 0x2000000; + .note : { *(.note) } :note + /DISCARD/ : { *(*) } +} diff --git a/ld/testsuite/ld-i386/alloc.d b/ld/testsuite/ld-i386/alloc.d new file mode 100644 index 0000000..c2b22f2 --- /dev/null +++ b/ld/testsuite/ld-i386/alloc.d @@ -0,0 +1,4 @@ +#name: Invalid allocated section +#as: --32 +#ld: -melf_i386 -T alloc.t +#error: .*section `.foo' can't be allocated in segment 0.* diff --git a/ld/testsuite/ld-i386/alloc.s b/ld/testsuite/ld-i386/alloc.s new file mode 100644 index 0000000..8c4f8fa --- /dev/null +++ b/ld/testsuite/ld-i386/alloc.s @@ -0,0 +1,6 @@ + .section .bar,"ax","progbits" + .byte 0 + .section .foo,"aw","progbits" + .byte 0 + .bss + .long 0 diff --git a/ld/testsuite/ld-i386/alloc.t b/ld/testsuite/ld-i386/alloc.t new file mode 100644 index 0000000..ea7f48c --- /dev/null +++ b/ld/testsuite/ld-i386/alloc.t @@ -0,0 +1,13 @@ +OUTPUT_FORMAT("elf32-i386", "elf32-i386", "elf32-i386") +OUTPUT_ARCH(i386) +PHDRS { + text PT_LOAD FLAGS(5); /* R_E */ +} +SECTIONS +{ + . = 0xC0000000 + ((0x100000 + (0x100000 - 1)) & ~(0x100000 - 1)); + .bar : AT(ADDR(.bar) - 0xC0000000) { *(.bar) } :text + .bss : AT(ADDR(.bss) - 0xC0000000) { *(.bss) } + .foo 0 : AT(ADDR(.bss) + SIZEOF(.bss) - 0xC0000000) { *(.foo) } :text + /DISCARD/ : { *(.*) } +} diff --git a/ld/testsuite/ld-i386/i386.exp b/ld/testsuite/ld-i386/i386.exp index d0d4cad..f008829 100644 --- a/ld/testsuite/ld-i386/i386.exp +++ b/ld/testsuite/ld-i386/i386.exp @@ -114,3 +114,4 @@ run_dump_test "abs" run_dump_test "pcrel8" run_dump_test "pcrel16" run_dump_test "pcrel16abs" +run_dump_test "alloc" -- 2.7.4