From: Nick Clifton Date: Fri, 1 Sep 2017 10:20:51 +0000 (+0100) Subject: Prevent an address violation parsing corrupt DWARF information by fixing the test... X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=7e8b60085eb3e6f2c41bc0c00c0d759fa7f72780;p=binutils-gdb.git Prevent an address violation parsing corrupt DWARF information by fixing the test for an overlong debug line info structure. PR 22059 * dwarf2.c (decode_line_info): Fix test for an overlong line info structure. --- diff --git a/bfd/ChangeLog b/bfd/ChangeLog index e0dd88fafad..8c6e8b14f75 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,9 @@ +2017-09-01 Nick Clifton + + PR 22059 + * dwarf2.c (decode_line_info): Fix test for an overlong line info + structure. + 2017-09-01 Nick Clifton PR 22058 diff --git a/bfd/dwarf2.c b/bfd/dwarf2.c index 40a187a1a43..856c963985c 100644 --- a/bfd/dwarf2.c +++ b/bfd/dwarf2.c @@ -2096,12 +2096,12 @@ decode_line_info (struct comp_unit *unit, struct dwarf2_debug *stash) offset_size = 8; } - if (lh.total_length > stash->dwarf_line_size) + if (unit->line_offset + lh.total_length > stash->dwarf_line_size) { _bfd_error_handler /* xgettext: c-format */ - (_("Dwarf Error: Line info data is bigger (%#Lx) than the section (%#Lx)"), - lh.total_length, stash->dwarf_line_size); + (_("Dwarf Error: Line info data is bigger (%#Lx) than the space remaining in the section (%#Lx)"), + lh.total_length, stash->dwarf_line_size - unit->line_offset); bfd_set_error (bfd_error_bad_value); return NULL; }