Skip to content

Commit f0554fc

Browse files
authored
Merge pull request #11155 from hvitved/ruby/avoid-stage-recomputation
Ruby: Avoid stage recomputation
2 parents edde3de + 37a69b4 commit f0554fc

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

ruby/ql/lib/codeql/ruby/regexp/internal/RegExpConfiguration.qll

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,7 @@ private import codeql.ruby.dataflow.internal.DataFlowImplForRegExp
77
private import codeql.ruby.typetracking.TypeTracker
88
private import codeql.ruby.ApiGraphs
99
private import codeql.ruby.dataflow.internal.DataFlowPrivate as DataFlowPrivate
10-
private import codeql.ruby.dataflow.internal.FlowSummaryImpl as FlowSummaryImpl
11-
private import codeql.ruby.dataflow.FlowSummary as FlowSummary
10+
private import codeql.ruby.TaintTracking
1211
private import codeql.ruby.frameworks.core.String
1312

1413
class RegExpConfiguration extends Configuration {
@@ -38,8 +37,8 @@ class RegExpConfiguration extends Configuration {
3837
}
3938

4039
override predicate isAdditionalFlowStep(DataFlow::Node nodeFrom, DataFlow::Node nodeTo) {
41-
// include taint flow through `String` summaries,
42-
FlowSummaryImpl::Private::Steps::summaryLocalStep(nodeFrom, nodeTo, false) and
40+
// include taint flow through `String` summaries
41+
TaintTracking::localTaintStep(nodeFrom, nodeTo) and
4342
nodeFrom.(DataFlowPrivate::SummaryNode).getSummarizedCallable() instanceof
4443
String::SummarizedCallable
4544
or

0 commit comments

Comments
 (0)