walkero
/
cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

ExprEngine: Robustness

This commit is contained in:
Daniel Marjamäki 2019-09-27 14:36:33 +02:00
parent 69016e38bc
commit 7ea8b69235
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
lib/exprengine.cpp
View File

@ -232,7 +232,10 @@ namespace {
s << "{"; // << dataIndex << ":"; s << "{"; // << dataIndex << ":";
for (auto mem : memory) { for (auto mem : memory) {
ExprEngine::ValuePtr value = mem.second; ExprEngine::ValuePtr value = mem.second;
s << " " << symbolDatabase->getVariableFromVarId(mem.first)->name() << "="; const Variable *var = symbolDatabase->getVariableFromVarId(mem.first);
if (!var)
continue;
s << " " << var->name() << "=";
if (!value) if (!value)
s << "(null)"; s << "(null)";
else if (value->name[0] == '$' && value->getSymbolicExpression() != value->name) else if (value->name[0] == '$' && value->getSymbolicExpression() != value->name)
@ -1002,7 +1005,11 @@ void ExprEngine::executeAllFunctions(const Tokenizer *tokenizer, const Settings
{ {
const SymbolDatabase *symbolDatabase = tokenizer->getSymbolDatabase(); const SymbolDatabase *symbolDatabase = tokenizer->getSymbolDatabase();
for (const Scope *functionScope : symbolDatabase->functionScopes) { for (const Scope *functionScope : symbolDatabase->functionScopes) {
executeFunction(functionScope, tokenizer, settings, callbacks); try {
executeFunction(functionScope, tokenizer, settings, callbacks);
} catch (const std::exception &) {
// FIXME.. there should not be exceptions
}
} }
} }
@ -1091,7 +1098,7 @@ void ExprEngine::executeFunction(const Scope *functionScope, const Tokenizer *to
void ExprEngine::runChecks(ErrorLogger *errorLogger, const Tokenizer *tokenizer, const Settings *settings) void ExprEngine::runChecks(ErrorLogger *errorLogger, const Tokenizer *tokenizer, const Settings *settings)
{ {
std::function<void(const Token *, const ExprEngine::Value &)> divByZero = [&](const Token *tok, const ExprEngine::Value &value) { std::function<void(const Token *, const ExprEngine::Value &)> divByZero = [&](const Token *tok, const ExprEngine::Value &value) {
if (!Token::simpleMatch(tok->astParent(), "/")) if (!Token::Match(tok->astParent(), "[/%]"))
return; return;
if (tok->astParent()->astOperand2() == tok && value.isIntValueInRange(0)) { if (tok->astParent()->astOperand2() == tok && value.isIntValueInRange(0)) {
std::list<const Token*> callstack{tok->astParent()}; std::list<const Token*> callstack{tok->astParent()};