X-Git-Url: http://drtracing.org/?a=blobdiff_plain;f=gprof%2Fcorefile.c;h=daf1315bece64f998a72059d5cc8a593e16b2d47;hb=5233f39b8b999f2675fb9493149e878c281e1d60;hp=e165da2c112286afb956d79c0751f90fb3708365;hpb=00927233079d1d65826fd611019e9167706b9ec6;p=deliverable%2Fbinutils-gdb.git diff --git a/gprof/corefile.c b/gprof/corefile.c index e165da2c11..daf1315bec 100644 --- a/gprof/corefile.c +++ b/gprof/corefile.c @@ -1,6 +1,6 @@ /* corefile.c - Copyright (C) 1999-2016 Free Software Foundation, Inc. + Copyright (C) 1999-2020 Free Software Foundation, Inc. This file is part of GNU Binutils. @@ -28,6 +28,7 @@ #include "hist.h" #include "corefile.h" #include "safe-ctype.h" +#include /* For UINT_MAX. */ bfd *core_bfd; static int core_num_syms; @@ -50,7 +51,6 @@ static bfd_boolean get_src_info extern void i386_find_call (Sym *, bfd_vma, bfd_vma); extern void alpha_find_call (Sym *, bfd_vma, bfd_vma); extern void vax_find_call (Sym *, bfd_vma, bfd_vma); -extern void tahoe_find_call (Sym *, bfd_vma, bfd_vma); extern void sparc_find_call (Sym *, bfd_vma, bfd_vma); extern void mips_find_call (Sym *, bfd_vma, bfd_vma); extern void aarch64_find_call (Sym *, bfd_vma, bfd_vma); @@ -186,6 +186,8 @@ core_init (const char * aout_name) done (1); } + core_bfd->flags |= BFD_DECOMPRESS; + if (!bfd_check_format (core_bfd, bfd_object)) { fprintf (stderr, _("%s: %s: not in executable format\n"), whoami, aout_name); @@ -249,7 +251,6 @@ core_init (const char * aout_name) switch (bfd_get_arch (core_bfd)) { case bfd_arch_vax: - case bfd_arch_tahoe: offset_to_code = 2; break; @@ -270,17 +271,17 @@ core_init (const char * aout_name) void core_get_text_space (bfd *cbfd) { - core_text_space = malloc (bfd_get_section_size (core_text_sect)); + core_text_space = malloc (bfd_section_size (core_text_sect)); if (!core_text_space) { fprintf (stderr, _("%s: ran out room for %lu bytes of text space\n"), - whoami, (unsigned long) bfd_get_section_size (core_text_sect)); + whoami, (unsigned long) bfd_section_size (core_text_sect)); done (1); } if (!bfd_get_section_contents (cbfd, core_text_sect, core_text_space, - 0, bfd_get_section_size (core_text_sect))) + 0, bfd_section_size (core_text_sect))) { bfd_perror ("bfd_get_section_contents"); free (core_text_space); @@ -318,10 +319,6 @@ find_call (Sym *parent, bfd_vma p_lowpc, bfd_vma p_highpc) sparc_find_call (parent, p_lowpc, p_highpc); break; - case bfd_arch_tahoe: - tahoe_find_call (parent, p_lowpc, p_highpc); - break; - case bfd_arch_mips: mips_find_call (parent, p_lowpc, p_highpc); break; @@ -500,7 +497,11 @@ num_of_syms_in (FILE * f) { if (sscanf (buf, "%" STR_BUFSIZE "s %c %" STR_BUFSIZE "s", address, &type, name) == 3) if (type == 't' || type == 'T') - ++num; + { + /* PR 20499 - prevent integer overflow computing argument to xmalloc. */ + if (++num >= UINT_MAX / sizeof (Sym)) + return -1U; + } } return num; @@ -531,11 +532,10 @@ core_create_syms_from (const char * sym_table_file) fprintf (stderr, _("%s: file `%s' has no symbols\n"), whoami, sym_table_file); done (1); } - /* PR 20499 - prevent integer overflow computing argument to xmalloc. */ - else if ((symtab.len * (unsigned) sizeof (Sym)) < symtab.len) + else if (symtab.len == -1U) { - fprintf (stderr, _("%s: file `%s' has too many symbols: %u\n"), - whoami, sym_table_file, symtab.len); + fprintf (stderr, _("%s: file `%s' has too many symbols\n"), + whoami, sym_table_file); done (1); } @@ -571,12 +571,6 @@ core_create_syms_from (const char * sym_table_file) max_vma = MAX (symtab.limit->addr, max_vma); ++symtab.limit; - /* PR 20499 - it is theoretically possible that there are so many - symbols in the file that the scan in num_of_syms_in() wrapped - around. So be paranoid here and exit the loop if we have - reached the end of our allocated table. */ - if ((unsigned int)(symtab.limit - symtab.base) == symtab.len) - break; } fclose (f); @@ -611,7 +605,6 @@ core_create_function_syms (void) case bfd_target_ecoff_flavour: case bfd_target_xcoff_flavour: case bfd_target_elf_flavour: - case bfd_target_nlm_flavour: case bfd_target_som_flavour: core_has_func_syms = 1; } @@ -682,7 +675,7 @@ core_create_function_syms (void) sym_sec = core_syms[i]->section; symtab.limit->addr = core_syms[i]->value; if (sym_sec) - symtab.limit->addr += bfd_get_section_vma (sym_sec->owner, sym_sec); + symtab.limit->addr += bfd_section_vma (sym_sec); if (found) { @@ -741,8 +734,8 @@ core_create_function_syms (void) section containing the symbol, if available. */ min_vma = MIN (symtab.limit->addr, min_vma); if (sym_sec) - max_vma = MAX (bfd_get_section_vma (sym_sec->owner, sym_sec) - + bfd_section_size (sym_sec->owner, sym_sec) - 1, + max_vma = MAX (bfd_section_vma (sym_sec) + + bfd_section_size (sym_sec) - 1, max_vma); else max_vma = MAX (symtab.limit->addr, max_vma); @@ -794,7 +787,7 @@ core_create_line_syms (void) ltab.len = 0; prev_line_num = 0; - vma_high = core_text_sect->vma + bfd_get_section_size (core_text_sect); + vma_high = core_text_sect->vma + bfd_section_size (core_text_sect); for (vma = core_text_sect->vma; vma < vma_high; vma += min_insn_size) { unsigned int len; @@ -849,7 +842,7 @@ core_create_line_syms (void) The old way called symtab_finalize before the is_static pass, causing a problem since symtab_finalize uses is_static as part of its address conflict resolution algorithm. Since global symbols - were prefered over static symbols, and all line symbols were + were preferred over static symbols, and all line symbols were global at that point, static function names that conflicted with their own line numbers (static, but labeled as global) were rejected in favor of the line num.