Skip to content

Commit 7ca6426

Browse files
committed
revert the Taint stage, as it caused an alert for ql/abstract-class-import
1 parent 7e4ab4c commit 7ca6426

File tree

2 files changed

+0
-31
lines changed

2 files changed

+0
-31
lines changed

python/ql/lib/semmle/python/dataflow/new/internal/TaintTrackingPrivate.qll

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ private import semmle.python.dataflow.new.DataFlow
33
private import semmle.python.dataflow.new.internal.DataFlowPrivate as DataFlowPrivate
44
private import semmle.python.dataflow.new.internal.TaintTrackingPublic
55
private import semmle.python.ApiGraphs
6-
private import semmle.python.internal.CachedStages
76

87
/**
98
* Holds if `node` should be a sanitizer in all global taint flow configurations
@@ -31,7 +30,6 @@ private module Cached {
3130
*/
3231
cached
3332
predicate defaultAdditionalTaintStep(DataFlow::Node nodeFrom, DataFlow::Node nodeTo) {
34-
Stages::Taint::ref() and
3533
localAdditionalTaintStep(nodeFrom, nodeTo)
3634
or
3735
any(AdditionalTaintStep a).step(nodeFrom, nodeTo)
@@ -44,7 +42,6 @@ private module Cached {
4442
*/
4543
cached
4644
predicate localAdditionalTaintStep(DataFlow::Node nodeFrom, DataFlow::Node nodeTo) {
47-
Stages::Taint::ref() and
4845
concatStep(nodeFrom, nodeTo)
4946
or
5047
subscriptStep(nodeFrom, nodeTo)

python/ql/lib/semmle/python/internal/CachedStages.qll

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -176,32 +176,4 @@ module Stages {
176176
PointsTo::AttributePointsTo::variableAttributePointsTo(_, _, _, _, _)
177177
}
178178
}
179-
180-
/**
181-
* The `taint` stage.
182-
*/
183-
cached
184-
module Taint {
185-
/**
186-
* Always holds.
187-
* Ensures that a predicate is evaluated as part of the DataFlow stage.
188-
*/
189-
cached
190-
predicate ref() { 1 = 1 }
191-
192-
private import semmle.python.dataflow.new.internal.TaintTrackingPrivate as TaintTrackingPrivate
193-
194-
/**
195-
* DONT USE!
196-
* Contains references to each predicate that use the above `ref` predicate.
197-
*/
198-
cached
199-
predicate backref() {
200-
1 = 1
201-
or
202-
TaintTrackingPrivate::localAdditionalTaintStep(_, _)
203-
or
204-
TaintTrackingPrivate::defaultAdditionalTaintStep(_, _)
205-
}
206-
}
207179
}

0 commit comments

Comments
 (0)