Fix getClassification crash

Author: glankk

lib/checkers.cpp

@@ -1901,6 +1901,14 @@ namespace checkers {
         {27,0,1,Req}
     };
 
+    const std::vector<MisraCppInfo> misraCpp2023Directives =
+    {
+        {0,3,1,Adv},
+        {0,3,2,Req},
+        {5,7,2,Adv},
+        {15,8,1,Req},
+    };
+
     const std::vector<MisraCppInfo> misraCpp2023Rules =
     {
         {0,0,1,Req},
@@ -1911,15 +1919,12 @@ namespace checkers {
         {0,2,2,Req},
         {0,2,3,Adv},
         {0,2,4,Adv},
-        {0,3,1,Adv},
-        {0,3,2,Req},
         {4,1,1,Req},
         {4,1,2,Adv},
         {4,1,3,Req},
         {4,6,1,Req},
         {5,0,1,Adv},
         {5,7,1,Req},
-        {5,7,2,Adv},
         {5,7,3,Req},
         {5,10,1,Req},
         {5,13,1,Req},
@@ -2025,7 +2030,6 @@ namespace checkers {
         {15,1,3,Req},
         {15,1,4,Adv},
         {15,1,5,Req},
-        {15,8,1,Req},
         {16,5,2,Req},
         {16,6,1,Adv},
         {17,8,1,Req},

lib/checkers.h

@@ -68,6 +68,7 @@ namespace checkers {
     extern CPPCHECKLIB const std::vector<MisraInfo> misraC2025Directives;
     extern CPPCHECKLIB const std::vector<MisraInfo> misraC2025Rules;
     extern CPPCHECKLIB const std::vector<MisraCppInfo> misraCpp2008Rules;
+    extern CPPCHECKLIB const std::vector<MisraCppInfo> misraCpp2023Directives;
     extern CPPCHECKLIB const std::vector<MisraCppInfo> misraCpp2023Rules;
 
     struct CPPCHECKLIB IdMapping {

lib/errorlogger.cpp

@@ -983,17 +983,22 @@ std::string getClassification(const std::string &guideline, ReportType reportTyp
     case ReportType::misraCpp2008:
     case ReportType::misraCpp2023:
     {
-        char delim;
         const std::vector<checkers::MisraCppInfo> *info;
+        std::vector<std::string> components;
+
         if (reportType == ReportType::misraCpp2008) {
-            delim = '-';
             info = &checkers::misraCpp2008Rules;
+            components = splitString(guideline, '-');
         } else {
-            delim = '.';
-            info = &checkers::misraCpp2023Rules;
+            if (guideline.rfind("Dir ", 0) == 0) {
+                components = splitString(guideline.substr(4), '.');
+                info = &checkers::misraCpp2023Directives;
+            } else {
+                components = splitString(guideline, '.');
+                info = &checkers::misraCpp2023Rules;
+            }
         }
 
-        auto components = splitString(guideline, delim);
         if (components.size() != 3)
             return "";
 

test/testerrorlogger.cpp

@@ -334,6 +334,8 @@ class TestErrorLogger : public TestFixture {
         testReportType(ReportType::misraC2012, Severity::error, "unusedVariable", "Advisory", "2.8");
         testReportType(ReportType::misraCpp2023, Severity::warning, "premium-misra-cpp-2023-6.8.4", "Advisory", "6.8.4");
         testReportType(ReportType::misraCpp2023, Severity::style, "premium-misra-cpp-2023-19.6.1", "Advisory", "19.6.1");
+        testReportType(ReportType::misraCpp2023, Severity::style, "premium-misra-cpp-2023-dir-0.3.1", "Advisory", "Dir 0.3.1");
+        testReportType(ReportType::misraCpp2023, Severity::style, "premium-misra-cpp-2023-dir-0.3.2", "Required", "Dir 0.3.2");
         testReportType(ReportType::misraCpp2008, Severity::style, "premium-misra-cpp-2008-3-4-1", "Required", "3-4-1");
         testReportType(ReportType::misraC2012, Severity::style, "premium-misra-c-2012-dir-4.6", "Advisory", "Dir 4.6");
         testReportType(ReportType::misraC2012, Severity::style, "misra-c2012-dir-4.6", "Advisory", "Dir 4.6");