gamzinav/cling
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed potential bug found by Coverity (https://coverity.cern.ch/reports.htm#v13142/p10001).

Browse Source
This commit is contained in:
Boris Perovic 2015-08-17 13:55:35 +02:00 committed by sftnight
parent 63bc59dde6
commit 34b8b73a30
1 changed files with 18 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
lib/Interpreter/ValuePrinter.cpp
View File

@ -225,21 +225,27 @@ static std::string printEnumValue(const Value &V){
std::stringstream enumString;
clang::ASTContext& C = V.getASTContext();
clang::QualType Ty = V.getType().getDesugaredType(C);
clang::EnumDecl* ED = Ty.getNonReferenceType()->getAs<clang::EnumType>()->getDecl();
uint64_t value = *(const uint64_t*)&V;
bool IsFirst = true;
llvm::APSInt ValAsAPSInt = C.MakeIntValue(value, Ty);
for (clang::EnumDecl::enumerator_iterator I = ED->enumerator_begin(),
E = ED->enumerator_end(); I != E; ++I) {
if (I->getInitVal() == ValAsAPSInt) {
if (!IsFirst) {
enumString << " ? ";
const clang::EnumType *EnumTy = Ty.getNonReferenceType()->getAs<clang::EnumType>();
if(EnumTy != 0) {
clang::EnumDecl *ED = EnumTy->getDecl();
uint64_t value = *(const uint64_t *) &V;
bool IsFirst = true;
llvm::APSInt ValAsAPSInt = C.MakeIntValue(value, Ty);
for (clang::EnumDecl::enumerator_iterator I = ED->enumerator_begin(),
E = ED->enumerator_end(); I != E; ++I) {
if (I->getInitVal() == ValAsAPSInt) {
if (!IsFirst) {
enumString << " ? ";
}
enumString << "(" << I->getQualifiedNameAsString() << ")";
IsFirst = false;
}
enumString << "(" << I->getQualifiedNameAsString() << ")";
IsFirst = false;
}
enumString << " : (int) " << ValAsAPSInt.toString(/*Radix = */10);
} else {
// Should not happen, we check for isEnumeralType before invoking this func
enumString << "ERROR: Value is not of enum type!";
}
enumString << " : (int) " << ValAsAPSInt.toString(/*Radix = */10);
return enumString.str();
}