BannedAPIs: Add tags

lcartey · lcartey · commit 6856e110e3c3 · 2025-07-07T15:39:46.000+01:00
diff --git a/cpp/misra/src/rules/RULE-18-5-2/AvoidProgramTerminatingFunctions.ql b/cpp/misra/src/rules/RULE-18-5-2/AvoidProgramTerminatingFunctions.ql
@@ -9,6 +9,8 @@
  * @problem.severity error
  * @tags external/misra/id/rule-18-5-2
  *       scope/single-translation-unit
+ *       maintainability
+ *       correctness
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/advisory
  */
diff --git a/cpp/misra/src/rules/RULE-21-10-1/NoVariadicFunctionMacros.ql b/cpp/misra/src/rules/RULE-21-10-1/NoVariadicFunctionMacros.ql
@@ -8,6 +8,8 @@
  * @problem.severity error
  * @tags external/misra/id/rule-21-10-1
  *       scope/single-translation-unit
+ *       correctness
+ *       maintainability
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/required
  */
diff --git a/cpp/misra/src/rules/RULE-21-10-2/NoCsetjmpHeader.ql b/cpp/misra/src/rules/RULE-21-10-2/NoCsetjmpHeader.ql
@@ -9,6 +9,8 @@
  * @problem.severity error
  * @tags external/misra/id/rule-21-10-2
  *       scope/single-translation-unit
+ *       correctness
+ *       maintainability
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/required
  */
diff --git a/cpp/misra/src/rules/RULE-21-2-2/UnsafeStringHandlingFunctions.ql b/cpp/misra/src/rules/RULE-21-2-2/UnsafeStringHandlingFunctions.ql
@@ -8,6 +8,8 @@
  * @problem.severity error
  * @tags external/misra/id/rule-21-2-2
  *       scope/single-translation-unit
+ *       correctness
+ *       maintainability
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/required
  */
diff --git a/cpp/misra/src/rules/RULE-21-2-3/BannedSystemFunction.ql b/cpp/misra/src/rules/RULE-21-2-3/BannedSystemFunction.ql
@@ -8,6 +8,9 @@
  * @problem.severity error
  * @tags external/misra/id/rule-21-2-3
  *       scope/single-translation-unit
+ *       correctness
+ *       maintainability
+ *       security
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/required
  */
diff --git a/cpp/misra/src/rules/RULE-23-11-1/UseSmartPtrFactoryFunctions.ql b/cpp/misra/src/rules/RULE-23-11-1/UseSmartPtrFactoryFunctions.ql
@@ -9,6 +9,8 @@
  * @problem.severity error
  * @tags external/misra/id/rule-23-11-1
  *       scope/single-translation-unit
+ *       correctness
+ *       maintainability
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/advisory
  */
diff --git a/cpp/misra/src/rules/RULE-24-5-1/CharacterHandlingFunctionRestrictions.ql b/cpp/misra/src/rules/RULE-24-5-1/CharacterHandlingFunctionRestrictions.ql
@@ -9,6 +9,8 @@
  * @problem.severity error
  * @tags external/misra/id/rule-24-5-1
  *       scope/single-translation-unit
+ *       correctness
+ *       maintainability
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/required
  */
diff --git a/cpp/misra/src/rules/RULE-24-5-2/NoMemoryFunctionsFromCString.ql b/cpp/misra/src/rules/RULE-24-5-2/NoMemoryFunctionsFromCString.ql
@@ -9,6 +9,8 @@
  * @problem.severity error
  * @tags external/misra/id/rule-24-5-2
  *       scope/single-translation-unit
+ *       correctness
+ *       maintainability
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/required
  */
diff --git a/cpp/misra/src/rules/RULE-25-5-1/LocaleGlobalFunctionNotAllowed.ql b/cpp/misra/src/rules/RULE-25-5-1/LocaleGlobalFunctionNotAllowed.ql
@@ -8,6 +8,9 @@
  * @problem.severity error
  * @tags external/misra/id/rule-25-5-1
  *       scope/single-translation-unit
+ *       correctness
+ *       maintainability
+ *       concurrency
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/required
  */
diff --git a/cpp/misra/src/rules/RULE-6-9-2/AvoidStandardIntegerTypeNames.ql b/cpp/misra/src/rules/RULE-6-9-2/AvoidStandardIntegerTypeNames.ql
@@ -8,6 +8,8 @@
  * @problem.severity error
  * @tags external/misra/id/rule-6-9-2
  *       scope/single-translation-unit
+ *       correctness
+ *       maintainability
  *       external/misra/enforcement/decidable
  *       external/misra/obligation/advisory
  */
diff --git a/rule_packages/cpp/BannedAPIs.json b/rule_packages/cpp/BannedAPIs.json
@@ -14,7 +14,9 @@
           "severity": "error",
           "short_name": "AvoidProgramTerminatingFunctions",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "maintainability",
+            "correctness"
           ]
         }
       ],
@@ -34,7 +36,9 @@
           "severity": "error",
           "short_name": "NoVariadicFunctionMacros",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "correctness",
+            "maintainability"
           ]
         }
       ],
@@ -54,7 +58,9 @@
           "severity": "error",
           "short_name": "NoCsetjmpHeader",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "correctness",
+            "maintainability"
           ]
         }
       ],
@@ -74,7 +80,9 @@
           "severity": "error",
           "short_name": "UnsafeStringHandlingFunctions",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "correctness",
+            "maintainability"
           ]
         }
       ],
@@ -94,7 +102,10 @@
           "severity": "error",
           "short_name": "BannedSystemFunction",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "correctness",
+            "maintainability",
+            "security"
           ]
         }
       ],
@@ -114,7 +125,9 @@
           "severity": "error",
           "short_name": "UseSmartPtrFactoryFunctions",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "correctness",
+            "maintainability"
           ]
         }
       ],
@@ -134,7 +147,9 @@
           "severity": "error",
           "short_name": "CharacterHandlingFunctionRestrictions",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "correctness",
+            "maintainability"
           ]
         }
       ],
@@ -154,7 +169,9 @@
           "severity": "error",
           "short_name": "NoMemoryFunctionsFromCString",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "correctness",
+            "maintainability"
           ]
         }
       ],
@@ -174,7 +191,10 @@
           "severity": "error",
           "short_name": "LocaleGlobalFunctionNotAllowed",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "correctness",
+            "maintainability",
+            "concurrency"
           ]
         }
       ],
@@ -195,7 +215,9 @@
           "short_name": "AvoidStandardIntegerTypeNames",
           "shared_implementation_short_name": "VariableWidthIntegerTypesUsed",
           "tags": [
-            "scope/single-translation-unit"
+            "scope/single-translation-unit",
+            "correctness",
+            "maintainability"
           ]
         }
       ],
