maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

ELF: Check StringRef::getAsInteger's return value. 

getAsInteger may partially update its result argument. We need to
check function return value to handle errors reliably.

llvm-svn: 227510
This commit is contained in:
Rui Ueyama 2015-01-29 23:23:57 +00:00
parent 0f776b0905
commit c1fa8300ec
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
lld/lib/ReaderWriter/ELF/ArrayOrderPass.cpp
View File

@ -42,13 +42,12 @@ void ArrayOrderPass::perform(std::unique_ptr<MutableFile> &f) {
// according to their number. Sections without optional
// numer suffix should go last.
uint32_t leftPriority = std::numeric_limits<uint32_t>::max();
if (!leftNum.empty())
leftNum.getAsInteger(10, leftPriority);
uint32_t rightPriority = std::numeric_limits<uint32_t>::max();
if (!rightNum.empty())
rightNum.getAsInteger(10, rightPriority);
uint32_t leftPriority;
uint32_t rightPriority;
if (leftNum.getAsInteger(10, leftPriority))
leftPriority = std::numeric_limits<uint32_t>::max();
if (rightNum.getAsInteger(10, rightPriority))
rightPriority = std::numeric_limits<uint32_t>::max();
return leftPriority < rightPriority;
});