Add check for npos, when checkInternal receives file with missing dot

Author: GNL10

lib/cppcheck.cpp

@@ -884,6 +884,27 @@ unsigned int CppCheck::checkFile(const FileWithDetails& file, const std::string
     return checkInternal(file, cfgname, fileIndex, f);
 }
 
+bool CppCheck::checkPlistOutput(const FileWithDetails& file, const std::vector<std::string>& files)
+{
+    if (!mSettings.plistOutput.empty()) {
+        const bool slashFound = file.spath().find('/') != std::string::npos;
+        std::string filename = slashFound ? file.spath().substr(file.spath().rfind('/') + 1) : file.spath();
+        const std::size_t dotPosition = filename.find('.');
+
+        if(dotPosition == std::string::npos) {
+            ErrorMessage::FileLocation loc(filename, 0, 0);
+            ErrorMessage errmsg({std::move(loc)}, "", Severity::error, "filename does not contain dot", "filenameError", Certainty::normal);
+            mErrorLogger.reportErr(errmsg);
+            return false;
+        }
+
+        const std::size_t fileNameHash = std::hash<std::string> {}(file.spath());
+        filename = mSettings.plistOutput + filename.substr(0, filename.find('.')) + "_" + std::to_string(fileNameHash) + ".plist";
+        mLogger->openPlist(filename, files);
+    }
+    return true;
+}
+
 unsigned int CppCheck::checkInternal(const FileWithDetails& file, const std::string &cfgname, int fileIndex, const CreateTokenListFn& createTokenList)
 {
     // TODO: move to constructor when CppCheck no longer owns the settings
@@ -989,16 +1010,8 @@ unsigned int CppCheck::checkInternal(const FileWithDetails& file, const std::str
         if (!preprocessor.loadFiles(tokens1, files))
             return mLogger->exitcode();
 
-        if (!mSettings.plistOutput.empty()) {
-            std::string filename2;
-            if (file.spath().find('/') != std::string::npos)
-                filename2 = file.spath().substr(file.spath().rfind('/') + 1);
-            else
-                filename2 = file.spath();
-            const std::size_t fileNameHash = std::hash<std::string> {}(file.spath());
-            filename2 = mSettings.plistOutput + filename2.substr(0, filename2.find('.')) + "_" + std::to_string(fileNameHash) + ".plist";
-            mLogger->openPlist(filename2, files);
-        }
+        if (!checkPlistOutput(file, files))
+            return mLogger->exitcode();
 
         std::string dumpProlog;
         if (mSettings.dump || !mSettings.addons.empty()) {

lib/cppcheck.h

@@ -185,6 +185,8 @@ class CPPCHECKLIB CppCheck {
      */
     unsigned int checkFile(const FileWithDetails& file, const std::string &cfgname, int fileIndex);
 
+    bool checkPlistOutput(const FileWithDetails& file, const std::vector<std::string>& files);
+
     /**
      * @brief Check a file using buffer
      * @param file the file

test/testcppcheck.cpp

@@ -79,6 +79,7 @@ class TestCppcheck : public TestFixture {
         TEST_CASE(isPremiumCodingStandardId);
         TEST_CASE(getDumpFileContentsRawTokens);
         TEST_CASE(getDumpFileContentsLibrary);
+        TEST_CASE(checkPlistOutput);
         TEST_CASE(premiumResultsCache);
         TEST_CASE(toomanyconfigs);
         TEST_CASE(purgedConfiguration);
@@ -522,6 +523,33 @@ class TestCppcheck : public TestFixture {
         }
     }
 
+    void checkPlistOutput() const {
+        Suppressions supprs;
+        ErrorLogger2 errorLogger;
+        std::vector<std::string> files = {"file.txt"};
+
+        {
+            const auto s = dinit(Settings, $.templateFormat = templateFormat, $.plistOutput = "output");
+            ScopedFile file("file", "");
+            CppCheck cppcheck(s, supprs, errorLogger, false, {});
+            ASSERT_EQUALS(false, cppcheck.checkPlistOutput(FileWithDetails(file.path(), Path::identify(file.path(), false), 0), files));
+        }
+
+        {
+            const auto s = dinit(Settings, $.plistOutput = "output");
+            ScopedFile file("file.c", "");
+            CppCheck cppcheck(s, supprs, errorLogger, false, {});
+            ASSERT_EQUALS(true, cppcheck.checkPlistOutput(FileWithDetails(file.path(), Path::identify(file.path(), false), 0), files));
+        }
+
+        {
+            Settings s;
+            ScopedFile file("file.c", "");
+            CppCheck cppcheck(s, supprs, errorLogger, false, {});
+            ASSERT_EQUALS(true, cppcheck.checkPlistOutput(FileWithDetails(file.path(), Path::identify(file.path(), false), 0), files));
+        }
+    }
+
     void premiumResultsCache() const {
         // Trac #13889 - cached misra results are shown after removing --premium=misra-c-2012 option