From af703f96203b5320baec426e5864cff71735a071 Mon Sep 17 00:00:00 2001 From: Jim Blandy Date: Thu, 5 Jul 2001 16:45:48 +0000 Subject: * dwarf2read (dwarf2_build_psymtabs_hard): Doc fix. --- gdb/ChangeLog | 4 ++++ gdb/dwarf2read.c | 13 +++++++++++++ 2 files changed, 17 insertions(+) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 4b2c4fc..da28812 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2001-07-05 Jim Blandy + + * dwarf2read (dwarf2_build_psymtabs_hard): Doc fix. + 2001-07-05 Mark Kettenis * config/i386/xm-go32.h (HOST_I386): Removed. diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index f398686..a3ca771 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -980,6 +980,19 @@ dwarf2_build_psymtabs_hard (struct objfile *objfile, int mainline) obstack_init (&dwarf2_tmp_obstack); back_to = make_cleanup (dwarf2_free_tmp_obstack, NULL); + /* Since the objects we're extracting from dwarf_info_buffer vary in + length, only the individual functions to extract them (like + read_comp_unit_head and read_partial_die) can really know whether + the buffer is large enough to hold another complete object. + + At the moment, they don't actually check that. If + dwarf_info_buffer holds just one extra byte after the last + compilation unit's dies, then read_comp_unit_head will happily + read off the end of the buffer. read_partial_die is similarly + casual. Those functions should be fixed. + + For this loop condition, simply checking whether there's any data + left at all should be sufficient. */ while (info_ptr < dwarf_info_buffer + dwarf_info_size) { struct comp_unit_head cu_header; -- cgit v1.1