Skip to content

Commit cc379b5

Browse files
authored
Merge pull request #19260 from smowton/smowton/feature/sanitize-enum-types
Java: Add EnumType to SimpleTypeSanitizer
2 parents 9498122 + 7a8dfdb commit cc379b5

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
---
2+
category: minorAnalysis
3+
---
4+
* Enum-typed values are now assumed to be safe by most queries. This means that queries may return fewer results where an enum value is used in a sensitive context, e.g. pasted into a query string.

java/ql/lib/semmle/code/java/security/Sanitizers.qll

+2-1
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ class SimpleTypeSanitizer extends DataFlow::Node {
2323
this.getType()
2424
.(RefType)
2525
.getASourceSupertype*()
26-
.hasQualifiedName("java.time.temporal", "TemporalAccessor")
26+
.hasQualifiedName("java.time.temporal", "TemporalAccessor") or
27+
this.getType() instanceof EnumType
2728
}
2829
}

0 commit comments

Comments
 (0)