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

src/checkbufferoverrun.cpp: fixed regression.

This commit is contained in:
Slava Semushin 2009-08-02 20:34:28 +07:00
parent 13532cbff7
commit be82d09003
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/checkbufferoverrun.cpp
View File

@ -255,6 +255,10 @@ void CheckBufferOverrun::checkScope(const Token *tok, const char *varname[], con
// Get index variable and stopsize. // Get index variable and stopsize.
const char *strindex = tok2->str().c_str(); const char *strindex = tok2->str().c_str();
bool condition_out_of_bounds = true;
int value = ((tok2->strAt(1)[1] == '=') ? 1 : 0) + std::atoi(tok2->strAt(2));
if (value <= size)
condition_out_of_bounds = false;;
// Goto the end of the for loop.. // Goto the end of the for loop..
while (tok2 && tok2->str() != ")") while (tok2 && tok2->str() != ")")
@ -287,7 +291,7 @@ void CheckBufferOverrun::checkScope(const Token *tok, const char *varname[], con
break; break;
} }
if (Token::Match(tok2, pattern.str().c_str())) if (Token::Match(tok2, pattern.str().c_str()) && condition_out_of_bounds)
{ {
bufferOverrun(tok2); bufferOverrun(tok2);
break; break;