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

Fixed Cppcheck warning about possible null pointer dereference

This commit is contained in:
Daniel Marjamäki 2015-07-08 13:41:27 +02:00
parent 23402b9b44
commit 5a9328f26a
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/checkother.cpp
View File

@ -306,12 +306,12 @@ void CheckOther::clarifyStatement()
const Scope * scope = symbolDatabase->functionScopes[i]; const Scope * scope = symbolDatabase->functionScopes[i];
for (const Token* tok = scope->classStart; tok && tok != scope->classEnd; tok = tok->next()) { for (const Token* tok = scope->classStart; tok && tok != scope->classEnd; tok = tok->next()) {
if (Token::Match(tok, "* %name%") && tok->astOperand1()) { if (Token::Match(tok, "* %name%") && tok->astOperand1()) {
const Token *tok2=tok->previous(); const Token *tok2 = tok->previous();
while (tok2 && tok2->str() == "*") while (tok2 && tok2->str() == "*")
tok2 = tok2->previous(); tok2 = tok2->previous();
if (!tok2->astParent() && Token::Match(tok2, "[{};]")) { if (tok2 && !tok2->astParent() && Token::Match(tok2, "[{};]")) {
tok2 = tok->astOperand1(); tok2 = tok->astOperand1();
if (Token::Match(tok2, "++|-- [;,]")) if (Token::Match(tok2, "++|-- [;,]"))
clarifyStatementError(tok2); clarifyStatementError(tok2);