fixed some missing --errorlist entries [skip ci]

firewave · firewave · commit 079a40ce1868 · 2025-06-03T11:45:07.000+02:00
diff --git a/lib/checkbufferoverrun.cpp b/lib/checkbufferoverrun.cpp
@@ -1235,4 +1235,7 @@ void CheckBufferOverrun::getErrorMessages(ErrorLogger *errorLogger, const Settin
     c.argumentSizeError(nullptr, "function", 1, "buffer", nullptr, nullptr);
     c.negativeMemoryAllocationSizeError(nullptr, nullptr);
     c.negativeArraySizeError(nullptr);
+    c.terminateStrncpyError(nullptr, "var_name");
+    // TODO: ctuArrayIndex
+    // TODO: ctuPointerArith
 }
diff --git a/lib/checkclass.cpp b/lib/checkclass.cpp
@@ -3814,4 +3814,5 @@ void CheckClass::getErrorMessages(ErrorLogger *errorLogger, const Settings *sett
     c.virtualFunctionCallInConstructorError(nullptr, std::list<const Token *>(), "f");
     c.thisUseAfterFree(nullptr, nullptr, nullptr);
     c.unsafeClassRefMemberError(nullptr, "UnsafeClass::var");
+    // TODO: ctuOneDefinitionRuleViolation
 }
diff --git a/lib/checkfunctions.cpp b/lib/checkfunctions.cpp
@@ -865,6 +865,7 @@ void CheckFunctions::getErrorMessages(ErrorLogger *errorLogger, const Settings *
     c.invalidFunctionArgBoolError(nullptr, "func_name", 1);
     c.invalidFunctionArgStrError(nullptr, "func_name", 1);
     c.ignoredReturnValueError(nullptr, "malloc");
+    c.ignoredReturnErrorCode(nullptr, "func_name");
     c.mathfunctionCallWarning(nullptr);
     c.mathfunctionCallWarning(nullptr, "1 - erf(x)", "erfc(x)");
     c.memsetZeroBytesError(nullptr);
@@ -873,4 +874,5 @@ void CheckFunctions::getErrorMessages(ErrorLogger *errorLogger, const Settings *
     c.missingReturnError(nullptr);
     c.copyElisionError(nullptr);
     c.useStandardLibraryError(nullptr, "memcpy");
+    // TODO: allocaCalled, <funcname>Called
 }
diff --git a/lib/checknullpointer.cpp b/lib/checknullpointer.cpp
@@ -719,5 +719,9 @@ void CheckNullPointer::getErrorMessages(ErrorLogger *errorLogger, const Settings
     CheckNullPointer c(nullptr, settings, errorLogger);
     c.nullPointerError(nullptr, "pointer", nullptr, false);
     c.pointerArithmeticError(nullptr, nullptr, false);
+    // TODO: nullPointerArithmeticOutOfMemory
     c.redundantConditionWarning(nullptr, nullptr, nullptr, false);
+    // TODO: ctunullpointer
+    // TODO: ctunullpointerOutOfMemory
+    // TODO: ctunullpointerOutOfResources
 }
diff --git a/lib/checkother.cpp b/lib/checkother.cpp
@@ -4580,6 +4580,8 @@ void CheckOther::getErrorMessages(ErrorLogger *errorLogger, const Settings *sett
     c.intToPointerCastError(nullptr, "decimal");
     c.suspiciousFloatingPointCastError(nullptr);
     c.passedByValueError(nullptr, false);
+    // TODO: iterateByValue
+    // TODO: passedByValueCallback
     c.constVariableError(nullptr, nullptr);
     c.constStatementError(nullptr, "type", false);
     c.signedCharArrayIndexError(nullptr);
@@ -4623,8 +4625,10 @@ void CheckOther::getErrorMessages(ErrorLogger *errorLogger, const Settings *sett
     c.knownArgumentError(nullptr, nullptr, nullptr, "x", false);
     c.knownPointerToBoolError(nullptr, nullptr);
     c.comparePointersError(nullptr, nullptr, nullptr);
+    // TODO: subtractPointers
     c.redundantAssignmentError(nullptr, nullptr, "var", false);
     c.redundantInitializationError(nullptr, nullptr, "var", false);
+    c.redundantContinueError(nullptr);
 
     const std::vector<const Token *> nullvec;
     c.funcArgOrderDifferent("function", nullptr, nullptr, nullvec, nullvec);
diff --git a/lib/checkstl.cpp b/lib/checkstl.cpp
@@ -3451,6 +3451,7 @@ void CheckStl::getErrorMessages(ErrorLogger* errorLogger, const Settings* settin
     c.iteratorsError(nullptr, nullptr, "container");
     c.invalidContainerLoopError(nullptr, nullptr, ErrorPath{});
     c.invalidContainerError(nullptr, nullptr, nullptr, ErrorPath{});
+    c.invalidContainerReferenceError(nullptr, nullptr, ErrorPath{});
     c.mismatchingContainerIteratorError(nullptr, nullptr, nullptr);
     c.mismatchingContainersError(nullptr, nullptr);
     c.mismatchingContainerExpressionError(nullptr, nullptr);
@@ -3465,19 +3466,26 @@ void CheckStl::getErrorMessages(ErrorLogger* errorLogger, const Settings* settin
     c.string_c_strError(nullptr);
     c.string_c_strReturn(nullptr);
     c.string_c_strParam(nullptr, 0);
+    c.string_c_strConstructor(nullptr);
+    c.string_c_strAssignment(nullptr);
+    c.string_c_strConcat(nullptr);
+    c.string_c_strStream(nullptr);
     c.string_c_strThrowError(nullptr);
     c.sizeError(nullptr);
     c.missingComparisonError(nullptr, nullptr);
     c.redundantIfRemoveError(nullptr);
     c.uselessCallsReturnValueError(nullptr, "str", "find");
     c.uselessCallsSwapError(nullptr, "str");
     c.uselessCallsSubstrError(nullptr, SubstrErrorType::COPY);
+    c.uselessCallsConstructorError(nullptr);
     c.uselessCallsEmptyError(nullptr);
     c.uselessCallsRemoveError(nullptr, "remove");
     c.dereferenceInvalidIteratorError(nullptr, "i");
+    // TODO: derefInvalidIteratorRedundantCheck
     c.eraseIteratorOutOfBoundsError(nullptr, nullptr);
     c.useStlAlgorithmError(nullptr, "");
     c.knownEmptyContainerError(nullptr, "");
     c.globalLockGuardError(nullptr);
     c.localMutexError(nullptr);
+    c.outOfBoundsIndexExpressionError(nullptr, nullptr);
 }
diff --git a/lib/checktype.cpp b/lib/checktype.cpp
@@ -540,7 +540,9 @@ void CheckType::getErrorMessages(ErrorLogger *errorLogger, const Settings *setti
     c.tooBigBitwiseShiftError(nullptr, 32, ValueFlow::Value(64));
     c.tooBigSignedBitwiseShiftError(nullptr, 31, ValueFlow::Value(31));
     c.integerOverflowError(nullptr, ValueFlow::Value(1LL<<32));
+    // TODO: integerOverflowCond
     c.signConversionError(nullptr, nullptr, false);
+    // TODO: signConversionCond
     c.longCastAssignError(nullptr);
     c.longCastReturnError(nullptr);
     ValueFlow::Value f;
diff --git a/lib/checkuninitvar.cpp b/lib/checkuninitvar.cpp
@@ -1813,7 +1813,7 @@ void CheckUninitVar::getErrorMessages(ErrorLogger* errorLogger, const Settings*
 
     ValueFlow::Value v{};
 
-    c.uninitvarError(nullptr, v);
+    c.uninitvarError(nullptr, v); // TODO: does not produce any output
     c.uninitdataError(nullptr, "varname");
     c.uninitStructMemberError(nullptr, "a.b");
 }
diff --git a/lib/checkunusedfunctions.cpp b/lib/checkunusedfunctions.cpp
@@ -339,11 +339,9 @@ static bool isOperatorFunction(const std::string & funcName)
     return std::find(additionalOperators.cbegin(), additionalOperators.cend(), funcName.substr(operatorPrefix.length())) != additionalOperators.cend();
 }
 
-static void staticFunctionError(ErrorLogger& errorLogger,
-                                const std::string &filename,
-                                unsigned int fileIndex,
-                                unsigned int lineNumber,
-                                const std::string &funcname)
+void CheckUnusedFunctions::staticFunctionError(ErrorLogger& errorLogger,
+                                               const std::string &filename, unsigned int fileIndex, unsigned int lineNumber,
+                                               const std::string &funcname)
 {
     std::list<ErrorMessage::FileLocation> locationList;
     if (!filename.empty()) {
diff --git a/lib/checkunusedfunctions.h b/lib/checkunusedfunctions.h
@@ -58,6 +58,7 @@ class CPPCHECKLIB CheckUnusedFunctions {
 
     static void getErrorMessages(ErrorLogger &errorLogger) {
         unusedFunctionError(errorLogger, "", 0, 0, "funcName");
+        staticFunctionError(errorLogger, "", 0, 0, "funcName");
     }
 
     // Return true if an error is reported.
@@ -70,6 +71,10 @@ class CPPCHECKLIB CheckUnusedFunctions {
                                     const std::string &filename, unsigned int fileIndex, unsigned int lineNumber,
                                     const std::string &funcname);
 
+    static void staticFunctionError(ErrorLogger& errorLogger,
+                                    const std::string &filename, unsigned int fileIndex, unsigned int lineNumber,
+                                    const std::string &funcname);
+
     struct CPPCHECKLIB FunctionUsage {
         std::string filename;
         unsigned int lineNumber{};
diff --git a/lib/cppcheck.cpp b/lib/cppcheck.cpp
@@ -35,6 +35,7 @@
 #include "settings.h"
 #include "standards.h"
 #include "suppressions.h"
+#include "symboldatabase.h"
 #include "timer.h"
 #include "token.h"
 #include "tokenize.h"
@@ -1933,6 +1934,7 @@ void CppCheck::getErrorMessages(ErrorLogger &errorlogger)
     CheckUnusedFunctions::getErrorMessages(errorlogger);
     Preprocessor::getErrorMessages(errorlogger, s);
     Tokenizer::getErrorMessages(errorlogger, s);
+    SymbolDatabase::getErrorMessages(errorlogger);
 }
 
 void CppCheck::analyseClangTidy(const FileSettings &fileSettings)
diff --git a/lib/symboldatabase.cpp b/lib/symboldatabase.cpp
@@ -8388,3 +8388,10 @@ ValueType::MatchResult ValueType::matchParameter(const ValueType *call, const Va
     }
     return res;
 }
+
+void SymbolDatabase::getErrorMessages(ErrorLogger&/*errorLogger*/)
+{
+    // TODO
+    //SymbolDatabase symdb;
+    //symdb.returnImplicitIntError(nullptr);
+}
diff --git a/lib/symboldatabase.h b/lib/symboldatabase.h
@@ -1420,6 +1420,8 @@ class CPPCHECKLIB SymbolDatabase {
     /* returns the opening { if tok points to enum */
     static const Token* isEnumDefinition(const Token* tok);
 
+    static void getErrorMessages(ErrorLogger &errorLogger);
+
 private:
     friend class Scope;
     friend class Function;

Original file line number	Diff line number	Diff line change
`@@ -1235,4 +1235,7 @@ void CheckBufferOverrun::getErrorMessages(ErrorLogger *errorLogger, const Settin`
`1235`	`1235`	`c.argumentSizeError(nullptr, "function", 1, "buffer", nullptr, nullptr);`
`1236`	`1236`	`c.negativeMemoryAllocationSizeError(nullptr, nullptr);`
`1237`	`1237`	`c.negativeArraySizeError(nullptr);`
	`1238`	`+ c.terminateStrncpyError(nullptr, "var_name");`
	`1239`	`+ // TODO: ctuArrayIndex`
	`1240`	`+ // TODO: ctuPointerArith`
`1238`	`1241`	`}`
Original file line number	Diff line number	Diff line change
`@@ -3814,4 +3814,5 @@ void CheckClass::getErrorMessages(ErrorLogger errorLogger, const Settings sett`
`3814`	`3814`	`c.virtualFunctionCallInConstructorError(nullptr, std::list<const Token *>(), "f");`
`3815`	`3815`	`c.thisUseAfterFree(nullptr, nullptr, nullptr);`
`3816`	`3816`	`c.unsafeClassRefMemberError(nullptr, "UnsafeClass::var");`
	`3817`	`+ // TODO: ctuOneDefinitionRuleViolation`
`3817`	`3818`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1813,7 +1813,7 @@ void CheckUninitVar::getErrorMessages(ErrorLogger* errorLogger, const Settings*`
`1813`	`1813`
`1814`	`1814`	`ValueFlow::Value v{};`
`1815`	`1815`
`1816`		`- c.uninitvarError(nullptr, v);`
	`1816`	`+ c.uninitvarError(nullptr, v); // TODO: does not produce any output`
`1817`	`1817`	`c.uninitdataError(nullptr, "varname");`
`1818`	`1818`	`c.uninitStructMemberError(nullptr, "a.b");`
`1819`	`1819`	`}`
Original file line number	Diff line number	Diff line change
`@@ -339,11 +339,9 @@ static bool isOperatorFunction(const std::string & funcName)`
`339`	`339`	`return std::find(additionalOperators.cbegin(), additionalOperators.cend(), funcName.substr(operatorPrefix.length())) != additionalOperators.cend();`
`340`	`340`	`}`
`341`	`341`
`342`		`-static void staticFunctionError(ErrorLogger& errorLogger,`
`343`		`- const std::string &filename,`
`344`		`- unsigned int fileIndex,`
`345`		`- unsigned int lineNumber,`
`346`		`- const std::string &funcname)`
	`342`	`+void CheckUnusedFunctions::staticFunctionError(ErrorLogger& errorLogger,`
	`343`	`+ const std::string &filename, unsigned int fileIndex, unsigned int lineNumber,`
	`344`	`+ const std::string &funcname)`
`347`	`345`	`{`
`348`	`346`	`std::list<ErrorMessage::FileLocation> locationList;`
`349`	`347`	`if (!filename.empty()) {`