Java: Add some medium precision queries to the code-quality-extended suite.

2025-12-16 16:53:25 +01:00 · 2025-09-09 13:10:37 +02:00
parent 046d0d4938
commit 56802035df
65 changed files with 185 additions and 96 deletions
--- a/java/ql/src/Advisory/Declarations/NonFinalImmutableField.ql
+++ b/java/ql/src/Advisory/Declarations/NonFinalImmutableField.ql
@@ -7,7 +7,10 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/non-final-immutable-field
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       correctness
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Declarations/NonPrivateField.ql
+++ b/java/ql/src/Advisory/Declarations/NonPrivateField.ql
@@ -6,7 +6,10 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/non-private-field
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
+ *       complexity
 */

 import java
--- a/java/ql/src/Advisory/Documentation/MissingJavadocMethods.ql
+++ b/java/ql/src/Advisory/Documentation/MissingJavadocMethods.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/undocumented-function
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Documentation/MissingJavadocParameters.ql
+++ b/java/ql/src/Advisory/Documentation/MissingJavadocParameters.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/undocumented-parameter
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Documentation/MissingJavadocReturnValues.ql
+++ b/java/ql/src/Advisory/Documentation/MissingJavadocReturnValues.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/undocumented-return-value
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Documentation/MissingJavadocThrows.ql
+++ b/java/ql/src/Advisory/Documentation/MissingJavadocThrows.ql
@@ -6,7 +6,10 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/undocumented-exception
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
+ *       error-handling
 */

 import java
--- a/java/ql/src/Advisory/Documentation/MissingJavadocTypes.ql
+++ b/java/ql/src/Advisory/Documentation/MissingJavadocTypes.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/undocumented-type
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/Objects/AvoidCloneMethodAccess.ql
+++ b/Objects/AvoidCloneMethodAccess.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/use-of-clone-method
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Objects/AvoidCloneOverride.ql
+++ b/Objects/AvoidCloneOverride.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/override-of-clone-method
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Objects/AvoidCloneableInterface.ql
+++ b/Objects/AvoidCloneableInterface.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/use-of-cloneable-interface
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Objects/AvoidFinalizeOverride.ql
+++ b/Objects/AvoidFinalizeOverride.ql
@@ -5,7 +5,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/override-of-finalize-method
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/java/ql/src/Advisory/Naming/NamingConventionsConstants.ql
+++ b/java/ql/src/Advisory/Naming/NamingConventionsConstants.ql
@@ -5,7 +5,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/misnamed-constant
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Naming/NamingConventionsMethods.ql
+++ b/java/ql/src/Advisory/Naming/NamingConventionsMethods.ql
@@ -5,7 +5,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/misnamed-function
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Naming/NamingConventionsPackages.ql
+++ b/java/ql/src/Advisory/Naming/NamingConventionsPackages.ql
@@ -5,7 +5,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/misnamed-package
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Naming/NamingConventionsRefTypes.ql
+++ b/java/ql/src/Advisory/Naming/NamingConventionsRefTypes.ql
@@ -5,7 +5,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/misnamed-type
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Naming/NamingConventionsVariables.ql
+++ b/java/ql/src/Advisory/Naming/NamingConventionsVariables.ql
@@ -5,7 +5,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/misnamed-variable
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Statements/MissingDefaultInSwitch.ql
+++ b/java/ql/src/Advisory/Statements/MissingDefaultInSwitch.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/missing-default-in-switch
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-478
 */

--- a/java/ql/src/Advisory/Statements/OneStatementPerLine.ql
+++ b/java/ql/src/Advisory/Statements/OneStatementPerLine.ql
@@ -5,7 +5,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/multiple-statements-on-same-line
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 */

 import java
--- a/java/ql/src/Advisory/Statements/TerminateIfElseIfWithElse.ql
+++ b/java/ql/src/Advisory/Statements/TerminateIfElseIfWithElse.ql
@@ -6,7 +6,9 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/non-terminated-if-else-if-chain
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/java/ql/src/Advisory/Types/GenericsConstructor.ql
+++ b/java/ql/src/Advisory/Types/GenericsConstructor.ql
@@ -6,7 +6,10 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/raw-constructor-invocation
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
+ *       correctness
 */

 import java
--- a/java/ql/src/Advisory/Types/GenericsReturnType.ql
+++ b/java/ql/src/Advisory/Types/GenericsReturnType.ql
@@ -6,7 +6,10 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/raw-return-type
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
+ *       correctness
 */

 import java
--- a/java/ql/src/Advisory/Types/GenericsVariable.ql
+++ b/java/ql/src/Advisory/Types/GenericsVariable.ql
@@ -6,7 +6,10 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/raw-variable
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
+ *       correctness
 */

 import java
--- a/java/ql/src/DeadCode/DeadClass.ql
+++ b/java/ql/src/DeadCode/DeadClass.ql
@@ -5,7 +5,8 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/dead-class
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-561
 */
--- a/java/ql/src/DeadCode/DeadEnumConstant.ql
+++ b/java/ql/src/DeadCode/DeadEnumConstant.ql
@@ -5,7 +5,8 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/dead-enum-constant
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-561
 */
--- a/java/ql/src/DeadCode/DeadField.ql
+++ b/java/ql/src/DeadCode/DeadField.ql
@@ -5,7 +5,8 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/dead-field
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-561
 */
--- a/java/ql/src/DeadCode/DeadMethod.ql
+++ b/java/ql/src/DeadCode/DeadMethod.ql
@@ -5,7 +5,8 @@
 * @problem.severity recommendation
 * @precision medium
 * @id java/dead-function
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       useless-code
 *       external/cwe/cwe-561
 */
--- a/Abuse/OverridePackagePrivate.ql
+++ b/Abuse/OverridePackagePrivate.ql
@@ -6,8 +6,10 @@
 * @problem.severity warning
 * @precision medium
 * @id java/non-overriding-package-private
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       readability
+ *       correctness
 */

 import java
--- a/Abuse/TypeVarExtendsFinalType.ql
+++ b/Abuse/TypeVarExtendsFinalType.ql
@@ -7,9 +7,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/type-bound-extends-final
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       readability
- *       types
 */

 import java
--- a/Bugs/Arithmetic/BadAbsOfRandom.ql
+++ b/Bugs/Arithmetic/BadAbsOfRandom.ql
@@ -6,8 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/abs-of-random
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Arithmetic/MultiplyRemainder.ql
+++ b/Bugs/Arithmetic/MultiplyRemainder.ql
@@ -6,7 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/multiplication-of-remainder
- * @tags maintainability
+ * @tags quality
+ *       maintainability
+ *       readability
 *       correctness
 */

--- a/Bugs/Arithmetic/RandomUsedOnce.ql
+++ b/Bugs/Arithmetic/RandomUsedOnce.ql
@@ -6,8 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/random-used-once
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-335
 */

--- a/Bugs/Cloning/MissingCallToSuperClone.ql
+++ b/Bugs/Cloning/MissingCallToSuperClone.ql
@@ -7,8 +7,9 @@
 * @problem.severity error
 * @precision medium
 * @id java/missing-call-to-super-clone
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-580
 */

--- a/Bugs/Cloning/MissingMethodClone.ql
+++ b/Bugs/Cloning/MissingMethodClone.ql
@@ -6,8 +6,9 @@
 * @problem.severity error
 * @precision medium
 * @id java/missing-clone-method
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Collections/IteratorRemoveMayFail.ql
+++ b/Bugs/Collections/IteratorRemoveMayFail.ql
@@ -6,9 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/iterator-remove-failure
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
- *       logic
 */

 import java
--- a/Bugs/Comparison/CovariantCompareTo.ql
+++ b/Bugs/Comparison/CovariantCompareTo.ql
@@ -6,7 +6,8 @@
 * @problem.severity error
 * @precision medium
 * @id java/wrong-compareto-signature
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 */

--- a/Bugs/Comparison/CovariantEquals.ql
+++ b/Bugs/Comparison/CovariantEquals.ql
@@ -6,7 +6,8 @@
 * @problem.severity error
 * @precision medium
 * @id java/wrong-equals-signature
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 */

--- a/Bugs/Comparison/InconsistentCompareTo.ql
+++ b/Bugs/Comparison/InconsistentCompareTo.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision medium
 * @id java/inconsistent-compareto-and-equals
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 */

--- a/Bugs/Comparison/StringComparison.ql
+++ b/Bugs/Comparison/StringComparison.ql
@@ -6,7 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/reference-equality-on-strings
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-597
 */

--- a/Bugs/Concurrency/CallsToConditionWait.ql
+++ b/Bugs/Concurrency/CallsToConditionWait.ql
@@ -6,7 +6,8 @@
 * @problem.severity error
 * @precision medium
 * @id java/wait-on-condition-interface
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       concurrency
 *       external/cwe/cwe-662
--- a/Bugs/Concurrency/DateFormatThreadUnsafe.ql
+++ b/Bugs/Concurrency/DateFormatThreadUnsafe.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision medium
 * @id java/thread-unsafe-dateformat
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       concurrency
 */
--- a/Bugs/Concurrency/FutileSynchOnField.ql
+++ b/Bugs/Concurrency/FutileSynchOnField.ql
@@ -6,7 +6,8 @@
 * @problem.severity error
 * @precision medium
 * @id java/unsafe-sync-on-field
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       concurrency
 *       language-features
--- a/Bugs/Concurrency/NotifyNotNotifyAll.ql
+++ b/Bugs/Concurrency/NotifyNotNotifyAll.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision medium
 * @id java/notify-instead-of-notify-all
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       concurrency
 *       external/cwe/cwe-662
--- a/Bugs/Concurrency/SleepWithLock.ql
+++ b/Bugs/Concurrency/SleepWithLock.ql
@@ -6,9 +6,11 @@
 * @problem.severity error
 * @precision medium
 * @id java/sleep-with-lock-held
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       concurrency
+ *       performance
 *       external/cwe/cwe-833
 */

--- a/Bugs/Concurrency/StartInConstructor.ql
+++ b/Bugs/Concurrency/StartInConstructor.ql
@@ -7,7 +7,8 @@
 * @problem.severity warning
 * @precision medium
 * @id java/thread-start-in-constructor
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       concurrency
 */
--- a/Bugs/Concurrency/SynchWriteObject.ql
+++ b/Bugs/Concurrency/SynchWriteObject.ql
@@ -6,10 +6,10 @@
 * @problem.severity warning
 * @precision medium
 * @id java/inconsistent-sync-writeobject
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 *       concurrency
- *       language-features
 *       external/cwe/cwe-662
 */

--- a/Bugs/Finalization/NullifiedSuperFinalize.ql
+++ b/Bugs/Finalization/NullifiedSuperFinalize.ql
@@ -6,8 +6,9 @@
 * @problem.severity error
 * @precision medium
 * @id java/missing-super-finalize
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-568
 */

--- a/Bugs/Frameworks/JUnit/BadSuiteMethod.ql
+++ b/Bugs/Frameworks/JUnit/BadSuiteMethod.ql
@@ -6,9 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/wrong-junit-suite-signature
- * @tags testability
- *       maintainability
- *       frameworks/junit
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Frameworks/Swing/BadlyOverriddenAdapter.ql
+++ b/Bugs/Frameworks/Swing/BadlyOverriddenAdapter.ql
@@ -7,9 +7,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/wrong-swing-event-adapter-signature
- * @tags reliability
- *       maintainability
- *       frameworks/swing
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Typos/DangerousNonCircuitLogic.ql
+++ b/Typos/DangerousNonCircuitLogic.ql
@@ -7,7 +7,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/non-short-circuit-evaluation
- * @tags reliability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       readability
 *       external/cwe/cwe-691
 */
--- a/Typos/EqualsTypo.ql
+++ b/Typos/EqualsTypo.ql
@@ -5,9 +5,10 @@
 * @problem.severity warning
 * @precision medium
 * @id java/equals-typo
- * @tags maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       readability
- *       naming
 */

 import java
--- a/Typos/HashCodeTypo.ql
+++ b/Typos/HashCodeTypo.ql
@@ -5,9 +5,10 @@
 * @problem.severity warning
 * @precision medium
 * @id java/hashcode-typo
- * @tags maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       readability
- *       naming
 */

 import java
--- a/Typos/ToStringTypo.ql
+++ b/Typos/ToStringTypo.ql
@@ -5,9 +5,10 @@
 * @problem.severity warning
 * @precision medium
 * @id java/tostring-typo
- * @tags maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       readability
- *       naming
 */

 import java
--- a/Bugs/Reflection/AnnotationPresentCheck.ql
+++ b/Bugs/Reflection/AnnotationPresentCheck.ql
@@ -6,8 +6,9 @@
 * @problem.severity error
 * @precision medium
 * @id java/ineffective-annotation-present-check
- * @tags correctness
- *       logic
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Serialization/IncorrectSerialVersionUID.ql
+++ b/Bugs/Serialization/IncorrectSerialVersionUID.ql
@@ -6,9 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/incorrect-serial-version-uid
- * @tags reliability
- *       maintainability
- *       language-features
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Serialization/IncorrectSerializableMethods.ql
+++ b/Bugs/Serialization/IncorrectSerializableMethods.ql
@@ -6,9 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/wrong-object-serialization-signature
- * @tags reliability
- *       maintainability
- *       language-features
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Serialization/MissingVoidConstructorOnExternalizable.ql
+++ b/Bugs/Serialization/MissingVoidConstructorOnExternalizable.ql
@@ -6,9 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/missing-no-arg-constructor-on-externalizable
- * @tags reliability
- *       maintainability
- *       language-features
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Serialization/MissingVoidConstructorsOnSerializable.ql
+++ b/Bugs/Serialization/MissingVoidConstructorsOnSerializable.ql
@@ -7,9 +7,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/missing-no-arg-constructor-on-serializable
- * @tags reliability
- *       maintainability
- *       language-features
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Serialization/NonSerializableInnerClass.ql
+++ b/Bugs/Serialization/NonSerializableInnerClass.ql
@@ -6,9 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/non-serializable-inner-class
- * @tags reliability
- *       maintainability
- *       language-features
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Serialization/ReadResolveObject.ql
+++ b/Bugs/Serialization/ReadResolveObject.ql
@@ -7,9 +7,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/wrong-readresolve-signature
- * @tags reliability
- *       maintainability
- *       language-features
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Bugs/Statements/MissingEnumInSwitch.ql
+++ b/Bugs/Statements/MissingEnumInSwitch.ql
@@ -6,8 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/missing-case-in-switch
- * @tags reliability
- *       readability
+ * @tags quality
+ *       reliability
+ *       correctness
 *       external/cwe/cwe-478
 */

--- a/Bugs/Termination/SpinOnField.ql
+++ b/Bugs/Termination/SpinOnField.ql
@@ -6,9 +6,11 @@
 * @problem.severity warning
 * @precision medium
 * @id java/spin-on-field
- * @tags efficiency
+ * @tags quality
+ *       reliability
 *       correctness
 *       concurrency
+ *       performance
 */

 import java
--- a/Code/CreatesEmptyZip.ql
+++ b/Code/CreatesEmptyZip.ql
@@ -6,8 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/empty-zip-file-entry
- * @tags reliability
- *       readability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Hiding/GetClassGetResource.ql
+++ b/Hiding/GetClassGetResource.ql
@@ -6,8 +6,9 @@
 * @problem.severity warning
 * @precision medium
 * @id java/unsafe-get-resource
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
 */

 import java
--- a/Conventions/FieldMasksSuperField.ql
+++ b/Conventions/FieldMasksSuperField.ql
@@ -7,8 +7,10 @@
 * @problem.severity warning
 * @precision medium
 * @id java/field-masks-super-field
- * @tags maintainability
+ * @tags quality
+ *       maintainability
 *       readability
+ *       correctness
 */

 import java
--- a/Calls/NextFromIterator.ql
+++ b/Calls/NextFromIterator.ql
@@ -6,7 +6,8 @@
 * @problem.severity warning
 * @precision medium
 * @id java/iterator-hasnext-calls-next
- * @tags reliability
+ * @tags quality
+ *       reliability
 *       correctness
 */