From 52d1fb02abf70ffbc7b6bfe17442c9e900fbc70a Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Thu, 12 May 2005 09:05:36 +0000 Subject: If pointer_size has not been found then assume that it is 4 in order to prevent a seg fault when process_extend_line_op attempts to read the line data. --- binutils/ChangeLog | 6 ++++++ binutils/readelf.c | 16 ++++++++++++---- 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/binutils/ChangeLog b/binutils/ChangeLog index ac2913a..00ffe8b 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,9 @@ +2005-05-12 Nick Clifton + + * readelf.c (display_debug_lines): If pointer_size has not been + found then assume that it is 4 in order to prevent a seg fault + when process_extend_line_op attempts to read the line data. + 2005-05-11 Alan Modra * readelf.c (get_ppc_dynamic_type): New function for DT_PPC_GLINK. diff --git a/binutils/readelf.c b/binutils/readelf.c index 9b83f08..c538dd8 100644 --- a/binutils/readelf.c +++ b/binutils/readelf.c @@ -7166,8 +7166,8 @@ reset_state_machine (int is_stmt) state_machine_regs.last_file_entry = 0; } -/* Handled an extend line op. Returns true if this is the end - of sequence. */ +/* Handled an extend line op. + Returns the number of bytes read. */ static int process_extended_line_op (unsigned char *data, int is_stmt, int pointer_size) @@ -9097,7 +9097,9 @@ display_debug_lines (Elf_Internal_Shdr *section, printf (_(" Line Base: %d\n"), info.li_line_base); printf (_(" Line Range: %d\n"), info.li_line_range); printf (_(" Opcode Base: %d\n"), info.li_opcode_base); - printf (_(" (Pointer size: %u)\n"), pointer_size); + printf (_(" (Pointer size: %u)%s\n"), + pointer_size, + warned_about_missing_comp_units ? " [assumed]" : "" ); end_of_sequence = data + info.li_length + initial_length_size; @@ -9188,8 +9190,14 @@ display_debug_lines (Elf_Internal_Shdr *section, else switch (op_code) { case DW_LNS_extended_op: + if (pointer_size == 0) + { + warn (_("Extend line ops need a valid pointer size, guessing at 4")); + pointer_size = 4; + } + data += process_extended_line_op (data, info.li_default_is_stmt, - pointer_size); + pointer_size); break; case DW_LNS_copy: -- cgit v1.1