From: Christian Eggers Date: Sun, 10 Mar 2019 18:21:57 +0000 (+0100) Subject: dwarf2: Pad size of .debug_line section. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=145c4477d239fef4e31a457ff8a1ba7153e9a448;p=binutils-gdb.git dwarf2: Pad size of .debug_line section. As all dwarf debug information is organized in octets, the size of all dwarf sections must be aligned to OCTETS_PER_BYTE. Most DWARF sections meet this requirement, only the .debug_line section can reach an arbitrary octet size. In order to align the size to a multiple of OCTETS_PER_BYTE, the section is padded with "nop" statements at the end. * dwarf2dbg.c (out_debug_line): Pad size of .debug_line section. --- diff --git a/gas/ChangeLog b/gas/ChangeLog index 368a73cb447..5b209741f15 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,7 @@ +2019-03-13 Christian Eggers + + * dwarf2dbg.c (out_debug_line): Pad size of .debug_line section. + 2019-03-13 Christian Eggers * dwarf2dbg.c (out_debug_str): Use octets for .debug_string pointers. diff --git a/gas/dwarf2dbg.c b/gas/dwarf2dbg.c index d4691385767..0b7b78c8c3a 100644 --- a/gas/dwarf2dbg.c +++ b/gas/dwarf2dbg.c @@ -1787,6 +1787,7 @@ out_debug_line (segT line_seg) symbolS *line_end; struct line_seg *s; int sizeof_offset; + addressT section_end, section_end_aligned; memset (&exp, 0, sizeof exp); sizeof_offset = out_header (line_seg, &exp); @@ -1848,6 +1849,16 @@ out_debug_line (segT line_seg) in the DWARF Line Number header. */ subseg_set (subseg_get (".debug_line_end", FALSE), 0); + /* Pad size of .debug_line section to a multiple of OCTETS_PER_BYTE. + Simply sizing the section in md_section_align() is not sufficient, + also the size field in the .debug_line header must be a multiple + of OCTETS_PER_BYTE. Not doing so will introduce gaps within the + .debug_line sections after linking. */ + section_end = frag_now_fix_octets (); + section_end_aligned = (section_end + OCTETS_PER_BYTE - 1) & -OCTETS_PER_BYTE; + for ( ; section_end != section_end_aligned; section_end++) + out_inc_line_addr (0, 0); /* NOP */ + symbol_set_value_now_octets (line_end); }