diff options
author | Richard Sandiford <rsandifo@nildram.co.uk> | 2002-06-06 09:24:56 +0000 |
---|---|---|
committer | Richard Sandiford <rsandifo@nildram.co.uk> | 2002-06-06 09:24:56 +0000 |
commit | 76f529a4faeb1dd3c3f74c244aabdd6173372cb9 (patch) | |
tree | efeaa308158a3fd1ee6e97bb787960204d9926ab /bfd/stabs.c | |
parent | 8fca799272e1948d4796d086d5ec589de6326229 (diff) | |
download | binutils-redhat-76f529a4faeb1dd3c3f74c244aabdd6173372cb9.tar.gz |
* stabs.c (_bfd_link_section_stabs): Check that the symbol offset
is within the .stabstr section.
Diffstat (limited to 'bfd/stabs.c')
-rw-r--r-- | bfd/stabs.c | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/bfd/stabs.c b/bfd/stabs.c index bba4a6d61e..e225d9cd60 100644 --- a/bfd/stabs.c +++ b/bfd/stabs.c @@ -284,6 +284,7 @@ _bfd_link_section_stabs (abfd, psinfo, stabsec, stabstrsec, psecinfo) sym < symend; sym += STABSIZE, ++pstridx) { + bfd_size_type symstroff; int type; const char *string; @@ -311,9 +312,18 @@ _bfd_link_section_stabs (abfd, psinfo, stabsec, stabstrsec, psecinfo) } /* Store the string in the hash table, and record the index. */ - string = ((char *) stabstrbuf - + stroff - + bfd_get_32 (abfd, sym + STRDXOFF)); + symstroff = stroff + bfd_get_32 (abfd, sym + STRDXOFF); + if (symstroff >= stabstrsec->_raw_size) + { + (*_bfd_error_handler) + (_("%s(%s+0x%lx): Stabs entry has invalid string index."), + bfd_archive_filename (abfd), + bfd_get_section_name (abfd, stabsec), + (long) (sym - stabbuf)); + bfd_set_error (bfd_error_bad_value); + goto error_return; + } + string = (char *) stabstrbuf + symstroff; *pstridx = _bfd_stringtab_add (sinfo->strings, string, true, true); /* An N_BINCL symbol indicates the start of the stabs entries |