Skip to content

Commit d4b10dd

Browse files
committed
C#
1 parent a212d4f commit d4b10dd

3 files changed

Lines changed: 13 additions & 15 deletions

File tree

csharp/ql/lib/semmle/code/csharp/dataflow/internal/DataFlowPrivate.qll

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -714,8 +714,7 @@ predicate simpleLocalFlowStep(Node nodeFrom, Node nodeTo, string model) {
714714
) and
715715
model = ""
716716
or
717-
FlowSummaryImpl::Private::Steps::summaryLocalStep(nodeFrom,
718-
nodeTo.(FlowSummaryNode).getSummaryNode(), true, model)
717+
FlowSummaryImpl::Private::Steps::summaryLocalStep(nodeFrom, nodeTo, true, model)
719718
}
720719

721720
/**
@@ -2085,8 +2084,7 @@ predicate jumpStep(Node pred, Node succ) {
20852084
)
20862085
)
20872086
or
2088-
FlowSummaryImpl::Private::Steps::summaryJumpStep(pred.(FlowSummaryNode).getSummaryNode(),
2089-
succ.(FlowSummaryNode).getSummaryNode())
2087+
FlowSummaryImpl::Private::Steps::summaryJumpStep(pred, succ)
20902088
or
20912089
succ = pred.(LocalFunctionCreationNode).getAnAccess(false)
20922090
}

csharp/ql/lib/semmle/code/csharp/dataflow/internal/FlowSummaryImpl.qll

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -137,15 +137,20 @@ private module Input2 implements Impl::Private::InputSig2 {
137137
DataFlowCallable getEnclosingCallable() { none() }
138138
}
139139

140-
bindingset[source, s]
141140
SourceSinkReportingElement getSourceEntryElement(
142-
Impl::Public::SourceElement source, Impl::Private::SummaryComponentStack s
141+
Impl::Public::SourceElement source, Impl::Private::SummaryComponent sc
143142
) {
144143
none()
145144
}
146145

147-
bindingset[e, s]
148-
Node getSourceExitNode(SourceSinkReportingElement e, Impl::Private::SummaryComponentStack s) {
146+
SourceSinkReportingElement getNextElement(
147+
SourceSinkReportingElement prev, Impl::Private::SummaryComponent sc
148+
) {
149+
none()
150+
}
151+
152+
bindingset[e, sc]
153+
Node getSourceExitNode(SourceSinkReportingElement e, Impl::Private::SummaryComponent sc) {
149154
none()
150155
}
151156

@@ -234,11 +239,7 @@ private module TypesInput implements Impl2::TypesInputSig {
234239
)
235240
}
236241

237-
DataFlowType getSourceType(Input::SourceBase source, Impl::Private::SummaryComponentStack s) {
238-
none()
239-
}
240-
241-
DataFlowType getSinkType(Input::SinkBase sink, Impl::Private::SummaryComponent sc) { none() }
242+
DataFlowType getSourceSinkType(Input2::SourceSinkReportingElement e) { none() }
242243
}
243244

244245
private module StepsInput implements Impl2::StepsInputSig {

csharp/ql/lib/semmle/code/csharp/dataflow/internal/TaintTrackingPrivate.qll

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -171,8 +171,7 @@ private module Cached {
171171
) and
172172
model = ""
173173
or
174-
FlowSummaryImpl::Private::Steps::summaryLocalStep(nodeFrom,
175-
nodeTo.(FlowSummaryNode).getSummaryNode(), false, model)
174+
FlowSummaryImpl::Private::Steps::summaryLocalStep(nodeFrom, nodeTo, false, model)
176175
}
177176
}
178177

0 commit comments

Comments
 (0)