Skip to content

Commit d1e662d

Browse files
authored
Merge pull request #10472 from erik-krogh/rename
JS: don't mention classes that don't exist in TaintTracking.qll
2 parents 6b9d546 + 58851ae commit d1e662d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

javascript/ql/lib/semmle/javascript/dataflow/TaintTracking.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -984,7 +984,7 @@ module TaintTracking {
984984
*
985985
* `<contains>` is one of: `contains`, `has`, `hasOwnProperty`
986986
*
987-
* Note that the `includes` method is covered by `StringInclusionSanitizer`.
987+
* Note that the `includes` method is covered by `MembershipTestSanitizer`.
988988
*/
989989
class WhitelistContainmentCallSanitizer extends AdditionalSanitizerGuardNode,
990990
DataFlow::MethodCallNode {
@@ -1171,7 +1171,7 @@ module TaintTracking {
11711171
/**
11721172
* A check of form `x.indexOf(y) > 0` or similar, which sanitizes `y` in the "then" branch.
11731173
*
1174-
* The more typical case of `x.indexOf(y) >= 0` is covered by `StringInclusionSanitizer`.
1174+
* The more typical case of `x.indexOf(y) >= 0` is covered by `MembershipTestSanitizer`.
11751175
*/
11761176
class PositiveIndexOfSanitizer extends AdditionalSanitizerGuardNode, DataFlow::ValueNode {
11771177
MethodCallExpr indexOf;

0 commit comments

Comments
 (0)