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

Fixed #5857 (strPlusChar: Crash on invalid code 'int+;')

This commit is contained in:
Daniel Marjamäki 2014-05-31 10:34:00 +02:00
parent 86dede14c0
commit 66ad3c97b9
2 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/checkother.cpp
View File

@ -2182,12 +2182,11 @@ void CheckOther::strPlusChar()
for (std::size_t i = 0; i < functions; ++i) {
const Scope * scope = symbolDatabase->functionScopes[i];
for (const Token* tok = scope->classStart->next(); tok != scope->classEnd; tok = tok->next()) {
if (tok->str() == "+" && tok->astOperand2()) {
if (tok->astOperand1()->type() == Token::eString) { // string literal...
if (tok->str() == "+") {
if (tok->astOperand1() && (tok->astOperand1()->type() == Token::eString)) { // string literal...
if (tok->astOperand2() && (tok->astOperand2()->type() == Token::eChar || isChar(tok->astOperand2()->variable()))) // added to char variable or char constant
strPlusCharError(tok);
}
}
}
}

5
test/testother.cpp
View File

@ -61,6 +61,7 @@ private:
TEST_CASE(strPlusChar1); // "/usr" + '/'
TEST_CASE(strPlusChar2); // "/usr" + ch
TEST_CASE(strPlusChar3); // ok: path + "/sub" + '/'
TEST_CASE(strPlusChar4); // ast
TEST_CASE(varScope1);
TEST_CASE(varScope2);
@ -901,6 +902,10 @@ private:
ASSERT_EQUALS("", errout.str());
}
void strPlusChar4() {
// don't crash
strPlusChar("int test() { int +; }");
}
void varScope(const char code[]) {