summaryrefslogtreecommitdiff
path: root/lib/Archive/ArchiveReader.cpp
diff options
context:
space:
mode:
authorReid Spencer <rspencer@reidspencer.com>2006-12-15 19:44:51 +0000
committerReid Spencer <rspencer@reidspencer.com>2006-12-15 19:44:51 +0000
commitcd5561a56e1521c7dc18744dcd371d255b580fdf (patch)
tree117b16301be70b3aff3ce54cf352c3028d48cf87 /lib/Archive/ArchiveReader.cpp
parentc2ff962bf97761a54c94d2dadb555060403d49c6 (diff)
downloadllvm-cd5561a56e1521c7dc18744dcd371d255b580fdf.tar.gz
llvm-cd5561a56e1521c7dc18744dcd371d255b580fdf.tar.bz2
llvm-cd5561a56e1521c7dc18744dcd371d255b580fdf.tar.xz
For PR1050:
Convert asserts into error messages. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@32607 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Archive/ArchiveReader.cpp')
-rw-r--r--lib/Archive/ArchiveReader.cpp13
1 files changed, 11 insertions, 2 deletions
diff --git a/lib/Archive/ArchiveReader.cpp b/lib/Archive/ArchiveReader.cpp
index e4769cac88..82ff9edc8a 100644
--- a/lib/Archive/ArchiveReader.cpp
+++ b/lib/Archive/ArchiveReader.cpp
@@ -68,7 +68,11 @@ Archive::parseSymbolTable(const void* data, unsigned size, std::string* error) {
ArchiveMember*
Archive::parseMemberHeader(const char*& At, const char* End, std::string* error)
{
- assert(At + sizeof(ArchiveMemberHeader) < End && "Not enough data");
+ if (At + sizeof(ArchiveMemberHeader) >= End) {
+ if (error)
+ *error = "Unexpected end of file";
+ return 0;
+ }
// Cast archive member header
ArchiveMemberHeader* Hdr = (ArchiveMemberHeader*)At;
@@ -498,7 +502,12 @@ Archive::findModulesDefiningSymbols(std::set<std::string>& symbols,
std::set<ModuleProvider*>& result,
std::string* error)
{
- assert(mapfile && base && "Can't findModulesDefiningSymbols on new archive");
+ if (!mapfile || !base) {
+ if (error)
+ *error = "Empty archive invalid for finding modules defining symbols";
+ return false;
+ }
+
if (symTab.empty()) {
// We don't have a symbol table, so we must build it now but lets also
// make sure that we populate the modules table as we do this to ensure