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

Fixed #2088 ([test.c:12]: (error) Buffer access out-of-bounds: l)

This commit is contained in:
Daniel Marjamäki 2010-10-12 19:35:20 +02:00
parent 4f0cda235a
commit 9e15c4ef38
2 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/checkbufferoverrun.cpp
View File

@ -1815,6 +1815,10 @@ bool CheckBufferOverrun::ArrayInfo::declare(const Token *tok, const Tokenizer &t
_element_size = tokenizer.sizeOfType(vartok); _element_size = tokenizer.sizeOfType(vartok);
vartok = vartok->next(); vartok = vartok->next();
} }
else if (tok->str() == "struct")
{
_element_size = 100;
}
else else
{ {
_element_size = tokenizer.sizeOfType(tok); _element_size = tokenizer.sizeOfType(tok);

10
test/testbufferoverrun.cpp
View File

@ -126,6 +126,7 @@ private:
TEST_CASE(buffer_overrun_13); TEST_CASE(buffer_overrun_13);
TEST_CASE(buffer_overrun_14); TEST_CASE(buffer_overrun_14);
TEST_CASE(buffer_overrun_15); // ticket #1787 TEST_CASE(buffer_overrun_15); // ticket #1787
TEST_CASE(buffer_overrun_16);
TEST_CASE(sprintf1); TEST_CASE(sprintf1);
TEST_CASE(sprintf2); TEST_CASE(sprintf2);
@ -1685,6 +1686,15 @@ private:
ASSERT_EQUALS("[test.cpp:7]: (error) Buffer access out-of-bounds\n", errout.str()); ASSERT_EQUALS("[test.cpp:7]: (error) Buffer access out-of-bounds\n", errout.str());
} }
void buffer_overrun_16()
{
check("void f() {\n"
" struct Foo foo[5];\n"
" memset(foo, 0, sizeof(foo));\n"
"}\n");
ASSERT_EQUALS("", errout.str());
}
void sprintf1() void sprintf1()
{ {
check("void f()\n" check("void f()\n"