This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] Don't call bfd_get_file_size in _bfd_generic_get_section_contents


Since it is impossible to read beyond the end of normal files, we only
need to check archive element size in _bfd_generic_get_section_contents

OK for master?

H.J.
---
	* libbfd.c (_bfd_generic_get_section_contents): Don't call
	bfd_get_file_size.  Check archive element size.
	(_bfd_generic_get_section_contents_in_window): Likewise.
---
 bfd/libbfd.c | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/bfd/libbfd.c b/bfd/libbfd.c
index 0d9de2b..82cffb8 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;
-- 
2.9.4


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]