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

Fixed #3517 and #3513

This commit is contained in:
PKEuS 2012-01-15 12:32:02 +01:00
parent c294b15360
commit 461565c50a
2 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/checkother.h
View File

@ -67,6 +67,7 @@ public:
checkOther.checkDuplicateExpression(); checkOther.checkDuplicateExpression();
checkOther.checkUnreachableCode(); checkOther.checkUnreachableCode();
checkOther.checkSuspiciousSemicolon(); checkOther.checkSuspiciousSemicolon();
checkOther.checkWrongPrintfScanfArguments();
// information checks // information checks
checkOther.checkVariableScope(); checkOther.checkVariableScope();
@ -91,7 +92,6 @@ public:
checkOther.checkCCTypeFunctions(); checkOther.checkCCTypeFunctions();
checkOther.checkFflushOnInputStream(); checkOther.checkFflushOnInputStream();
checkOther.invalidScanf(); checkOther.invalidScanf();
checkOther.checkWrongPrintfScanfArguments();
checkOther.checkCoutCerrMisusage(); checkOther.checkCoutCerrMisusage();
checkOther.checkIncorrectLogicOperator(); checkOther.checkIncorrectLogicOperator();

13
test/testother.cpp
View File

@ -2027,9 +2027,9 @@ private:
" fclose(file);\n" " fclose(file);\n"
" return b;\n" " return b;\n"
"}"); "}");
ASSERT_EQUALS("[test.cpp:6]: (warning) scanf without field width limits can crash with huge input data\n" ASSERT_EQUALS("[test.cpp:8]: (warning) fscanf format string has 0 parameters but 1 are given\n"
"[test.cpp:7]: (warning) scanf without field width limits can crash with huge input data\n" "[test.cpp:6]: (warning) scanf without field width limits can crash with huge input data\n"
"[test.cpp:8]: (warning) fscanf format string has 0 parameters but 1 are given\n", errout.str()); "[test.cpp:7]: (warning) scanf without field width limits can crash with huge input data\n", errout.str());
} }
void testScanf2() { void testScanf2() {
@ -2044,9 +2044,9 @@ private:
" fclose(file);\n" " fclose(file);\n"
" return b;\n" " return b;\n"
"}"); "}");
ASSERT_EQUALS("[test.cpp:6]: (warning) scanf without field width limits can crash with huge input data\n" ASSERT_EQUALS("[test.cpp:8]: (warning) fscanf format string has 0 parameters but 1 are given\n"
"[test.cpp:7]: (warning) scanf without field width limits can crash with huge input data\n" "[test.cpp:6]: (warning) scanf without field width limits can crash with huge input data\n"
"[test.cpp:8]: (warning) fscanf format string has 0 parameters but 1 are given\n", errout.str()); "[test.cpp:7]: (warning) scanf without field width limits can crash with huge input data\n", errout.str());
} }
void testScanf3() { void testScanf3() {
@ -2212,6 +2212,7 @@ private:
" printf(\"%G\", bp);\n" " printf(\"%G\", bp);\n"
" printf(\"%f\", d);\n" " printf(\"%f\", d);\n"
" printf(\"%f\", b);\n" " printf(\"%f\", b);\n"
" printf(\"%f\", (float)cpi);\n"
"}"); "}");
ASSERT_EQUALS("[test.cpp:3]: (warning) %e in format string (no. 1) requires a floating point number given in the argument list\n" ASSERT_EQUALS("[test.cpp:3]: (warning) %e in format string (no. 1) requires a floating point number given in the argument list\n"
"[test.cpp:4]: (warning) %E in format string (no. 1) requires a floating point number given in the argument list\n" "[test.cpp:4]: (warning) %E in format string (no. 1) requires a floating point number given in the argument list\n"