Skip to content

Commit 83a9b0e

Browse files
Apply suggestions from code review
Co-authored-by: Anders Schack-Mulligen <[email protected]>
1 parent 893f84f commit 83a9b0e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

java/ql/src/semmle/code/java/frameworks/Jackson.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ class SetPolymorphicTypeValidatorSource extends DataFlow::ExprNode {
7777
}
7878
}
7979

80-
/** Holds if `fromNode` to `toNode` is a dataflow step that resolves a class. s */
80+
/** Holds if `fromNode` to `toNode` is a dataflow step that resolves a class. */
8181
predicate resolveClassStep(DataFlow::Node fromNode, DataFlow::Node toNode) {
8282
exists(ReflectiveClassIdentifierMethodAccess ma |
8383
ma.getArgument(0) = fromNode.asExpr() and
@@ -88,7 +88,7 @@ predicate resolveClassStep(DataFlow::Node fromNode, DataFlow::Node toNode) {
8888
/**
8989
* Holds if `fromNode` to `toNode` is a dataflow step that creates a Jackson parser.
9090
*
91-
* For example, a `createParser(userString)` call yields a `JsonParser` which becomes dangerous
91+
* For example, a `createParser(userString)` call yields a `JsonParser`, which becomes dangerous
9292
* if passed to an unsafely-configured `ObjectMapper`'s `readValue` method.
9393
*/
9494
predicate createJacksonJsonParserStep(DataFlow::Node fromNode, DataFlow::Node toNode) {

0 commit comments

Comments
 (0)