diff --git a/lib/checkbufferoverrun.cpp b/lib/checkbufferoverrun.cpp index e794e538f..2f7b1e2ac 100644 --- a/lib/checkbufferoverrun.cpp +++ b/lib/checkbufferoverrun.cpp @@ -779,17 +779,17 @@ void CheckBufferOverrun::checkScope(const Token *tok, const ArrayInfo &arrayInfo } } - - // in case %var% is declared as a pointer - else if (Token::Match(tok, "%var% [ %num% ]")) - { - const int index = MathLib::toLongNumber(tok->strAt(2)); - if (index < 0) - { - arrayIndexOutOfBounds(tok, index, index); - } - - } + + // in case %var% is declared as a pointer + else if (Token::Match(tok, "%var% [ %num% ]")) + { + const int index = MathLib::toLongNumber(tok->strAt(2)); + if (index < 0) + { + arrayIndexOutOfBounds(tok, index, index); + } + + } // Loop.. else if (Token::simpleMatch(tok, "for (")) diff --git a/test/testbufferoverrun.cpp b/test/testbufferoverrun.cpp index 0ebda2a2f..ee90a24e0 100644 --- a/test/testbufferoverrun.cpp +++ b/test/testbufferoverrun.cpp @@ -933,17 +933,17 @@ private: "}\n"); ASSERT_EQUALS("[test.cpp:5]: (error) Array 'a[10]' index -1 out of bounds\n", errout.str()); } - - void array_index_28() + + void array_index_28() { - // ticket #1418 + // ticket #1418 check("void f()\n" "{\n" " int i[2];\n" " int *ip = &i[1];\n" " ip[-10] = 1;\n" "}\n"); - ASSERT_EQUALS("[test.cpp:5]: (error) Array 'ip[-10]' index -10 out of bounds\n", errout.str()); + ASSERT_EQUALS("[test.cpp:5]: (error) Array 'ip[-10]' index -10 out of bounds\n", errout.str()); } void array_index_multidim()