/* corefile.c
- Copyright (C) 1999-2016 Free Software Foundation, Inc.
+ Copyright (C) 1999-2017 Free Software Foundation, Inc.
This file is part of GNU Binutils.
#include "hist.h"
#include "corefile.h"
#include "safe-ctype.h"
+#include <limits.h> /* For UINT_MAX. */
bfd *core_bfd;
static int core_num_syms;
{
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;
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);
}
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);
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.