diff options
author | Tristan Gingold <gingold@adacore.com> | 2011-06-09 15:29:05 +0000 |
---|---|---|
committer | Tristan Gingold <gingold@adacore.com> | 2011-06-09 15:29:05 +0000 |
commit | 4c23495ec3cf3e931b89724441a83fc9457a4c14 (patch) | |
tree | 602cb6877dc3a6e2730e20481072641be54e3d8b | |
parent | 7ab1d000c11675aaea3b078e3e2a6cd3f1beaa08 (diff) | |
download | binutils-redhat-4c23495ec3cf3e931b89724441a83fc9457a4c14.tar.gz |
2011-06-09 Tristan Gingold <gingold@adacore.com>
* od-xcoff.c (xcoff32_read_symbols): Allow missing string table
length.
-rw-r--r-- | binutils/ChangeLog | 5 | ||||
-rw-r--r-- | binutils/od-xcoff.c | 9 |
2 files changed, 8 insertions, 6 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 0de33e513b..918c7e1def 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,8 @@ +2011-06-09 Tristan Gingold <gingold@adacore.com> + + * od-xcoff.c (xcoff32_read_symbols): Allow missing string table + length. + 2011-06-08 Nick Clifton <nickc@redhat.com> PR binutils/12855 diff --git a/binutils/od-xcoff.c b/binutils/od-xcoff.c index 47d0248847..5b8b5896eb 100644 --- a/binutils/od-xcoff.c +++ b/binutils/od-xcoff.c @@ -639,17 +639,14 @@ xcoff32_read_symbols (bfd *abfd, struct xcoff_dump *data) + data->nsyms * (unsigned)sizeof (struct external_syment); /* Read string table. */ - if (bfd_seek (abfd, stptr, SEEK_SET) != 0) + if (bfd_seek (abfd, stptr, SEEK_SET) != 0 + || bfd_bread (&stsz_arr, sizeof (stsz_arr), abfd) != sizeof (stsz_arr)) { + non_fatal (_("cannot read strings table length")); data->strings_size = 0; } else { - if (bfd_bread (&stsz_arr, sizeof (stsz_arr), abfd) != sizeof (stsz_arr)) - { - non_fatal (_("cannot read strings table len")); - return; - } data->strings_size = bfd_h_get_32 (abfd, stsz_arr); if (data->strings_size > sizeof (stsz_arr)) { |