diff options
| -rw-r--r-- | bfd/libbfd.c | 25 |
1 files changed, 14 insertions, 11 deletions
diff --git a/bfd/libbfd.c b/bfd/libbfd.c index 0d9de2b30bd..82cffb865f8 100644 --- a/bfd/libbfd.c +++ b/bfd/libbfd.c @@ -788,8 +788,7 @@ _bfd_generic_get_section_contents (bfd *abfd, file_ptr offset, bfd_size_type count) { - bfd_size_type sz; - ufile_ptr filesz; + bfd_size_type sz, end; if (count == 0) return TRUE; @@ -812,10 +811,12 @@ _bfd_generic_get_section_contents (bfd *abfd, sz = section->rawsize; else sz = section->size; - filesz = bfd_get_file_size (abfd); - if (offset + count < count - || offset + count > sz - || (ufile_ptr) section->filepos + offset + count > filesz) + end = offset + count; + if (end < count + || end > sz + || (abfd->my_archive != NULL + && !bfd_is_thin_archive (abfd->my_archive) + && (ufile_ptr) section->filepos + end > arelt_size (abfd))) { bfd_set_error (bfd_error_invalid_operation); return FALSE; @@ -837,8 +838,7 @@ _bfd_generic_get_section_contents_in_window bfd_size_type count ATTRIBUTE_UNUSED) { #ifdef USE_MMAP - bfd_size_type sz; - ufile_ptr filesz; + bfd_size_type sz, end; if (count == 0) return TRUE; @@ -871,9 +871,12 @@ _bfd_generic_get_section_contents_in_window sz = section->rawsize; else sz = section->size; - filesz = bfd_get_file_size (abfd); - if (offset + count > sz - || (ufile_ptr) section->filepos + offset + count > filesz + end = offset + count; + if (end < count + || end > sz + || (abfd->my_archive != NULL + && !bfd_is_thin_archive (abfd->my_archive) + && (ufile_ptr) section->filepos + end > arelt_size (abfd)) || ! bfd_get_file_window (abfd, section->filepos + offset, count, w, TRUE)) return FALSE; |