Skip to content

Commit 8b060c4

Browse files
committed
JS: Remove TODO about evaluating legacy steps
There is an issue for tracking this. It's not a small fix.
1 parent a8f93ca commit 8b060c4

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

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

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -232,10 +232,7 @@ class LegacyTaintStep extends Unit {
232232
cached
233233
private module Cached {
234234
cached
235-
predicate forceStage() {
236-
// TODO: ensure that this stage is only evaluated if using the old data flow library
237-
Stages::Taint::ref()
238-
}
235+
predicate forceStage() { Stages::Taint::ref() }
239236

240237
/**
241238
* Holds if `pred` → `succ` should be considered a taint-propagating

0 commit comments

Comments
 (0)