diff options
| author | Cary Coutant <ccoutant@google.com> | 2008-09-10 17:56:02 +0000 |
|---|---|---|
| committer | Cary Coutant <ccoutant@google.com> | 2008-09-10 17:56:02 +0000 |
| commit | a9caad024e20635ecebb7be5330747fe23a7292b (patch) | |
| tree | 6cfb7c02315baa1e27e8d96ce2e6acd5333f908e /gold | |
| parent | ebcf6f00fcd8bd37721863ca65290c9b28a13d60 (diff) | |
| download | binutils-a9caad024e20635ecebb7be5330747fe23a7292b.zip binutils-a9caad024e20635ecebb7be5330747fe23a7292b.tar.gz binutils-a9caad024e20635ecebb7be5330747fe23a7292b.tar.bz2 | |
2008-09-05 Cary Coutant <ccoutant@google.com>
* fileread.cc (File_read::make_view): Add check for attempt to map
beyond end of file.
Diffstat (limited to 'gold')
| -rw-r--r-- | gold/ChangeLog | 5 | ||||
| -rw-r--r-- | gold/fileread.cc | 8 |
2 files changed, 13 insertions, 0 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog index cac87e6..1df7d74 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,5 +1,10 @@ 2008-09-05 Cary Coutant <ccoutant@google.com> + * fileread.cc (File_read::make_view): Add check for attempt to map + beyond end of file. + +2008-09-05 Cary Coutant <ccoutant@google.com> + * symtab.cc (Symbol_table::add_from_dynobj): Fix typos in explicit instantiations. diff --git a/gold/fileread.cc b/gold/fileread.cc index 8c2f8f3..f56e33e 100644 --- a/gold/fileread.cc +++ b/gold/fileread.cc @@ -355,6 +355,14 @@ File_read::make_view(off_t start, section_size_type size, { gold_assert(size > 0); + // Check that start and end of the view are within the file. + if (start > this->size_ || size > this->size_ - start) + gold_fatal(_("%s: attempt to map %lld bytes at offset %lld exceeds " + "size of file; the file may be corrupt"), + this->filename().c_str(), + static_cast<long long>(size), + static_cast<long long>(start)); + off_t poff = File_read::page_offset(start); section_size_type psize = File_read::pages(size + (start - poff)); |