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

Make sure we have a valid Variable pointer before dereferencing it 

All other call sites check the pointer already.
This commit is contained in:
Thomas Jarosch 2015-01-14 22:43:23 +01:00
parent 823af34c67
commit ef7f104335
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/checkunusedvar.cpp
View File

@ -1119,7 +1119,7 @@ void CheckUnusedVar::checkFunctionVariableUsage()
unusedVariableError(usage._var->nameToken(), varname);
// variable has not been written but has been modified
else if (usage._modified && !usage._write && !usage._allocateMemory && !var->isStlType())
else if (usage._modified && !usage._write && !usage._allocateMemory && var && !var->isStlType())
unassignedVariableError(usage._var->nameToken(), varname);
// variable has been written but not read
@ -1127,7 +1127,7 @@ void CheckUnusedVar::checkFunctionVariableUsage()
unreadVariableError(usage._lastAccess, varname);
// variable has been read but not written
else if (!usage._write && !usage._allocateMemory && !var->isStlType() && !isEmptyType(var->type()))
else if (!usage._write && !usage._allocateMemory && !var->isStlType() && var && !isEmptyType(var->type()))
unassignedVariableError(usage._var->nameToken(), varname);
}
}