Skip to content

Commit 262f64f

Browse files
committed
Dataflow: Remove unused columns.
1 parent d404406 commit 262f64f

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

shared/dataflow/codeql/dataflow/internal/DataFlowImpl.qll

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1562,7 +1562,7 @@ module MakeImpl<LocationSig Location, InputSig<Location> Lang> {
15621562
or
15631563
// flow through a callable
15641564
exists(DataFlowCall call, CcCall ccc, RetNodeEx ret, boolean allowsFieldFlow |
1565-
fwdFlowThrough(call, cc, state, ccc, summaryCtx, t, ap, apa, stored, ret, _) and
1565+
fwdFlowThrough(call, cc, state, ccc, summaryCtx, t, ap, apa, stored, ret) and
15661566
flowThroughOutOfCall(call, ccc, ret, node, allowsFieldFlow) and
15671567
not inBarrier(node, state) and
15681568
if allowsFieldFlow = false then ap instanceof ApNil else any()
@@ -2098,10 +2098,9 @@ module MakeImpl<LocationSig Location, InputSig<Location> Lang> {
20982098
pragma[nomagic]
20992099
private predicate fwdFlowThrough(
21002100
DataFlowCall call, Cc cc, FlowState state, CcCall ccc, SummaryCtx summaryCtx, Typ t,
2101-
Ap ap, ApApprox apa, TypOption stored, RetNodeEx ret, ApApprox innerArgApa
2101+
Ap ap, ApApprox apa, TypOption stored, RetNodeEx ret
21022102
) {
2103-
fwdFlowThrough0(call, _, cc, state, ccc, summaryCtx, t, ap, apa, stored, ret, _,
2104-
innerArgApa)
2103+
fwdFlowThrough0(call, _, cc, state, ccc, summaryCtx, t, ap, apa, stored, ret, _, _)
21052104
}
21062105

21072106
pragma[nomagic]
@@ -3136,10 +3135,10 @@ module MakeImpl<LocationSig Location, InputSig<Location> Lang> {
31363135
private predicate fwdFlowThroughStep0(
31373136
DataFlowCall call, ArgNodeEx arg, Cc cc, FlowState state, CcCall ccc,
31383137
SummaryCtx summaryCtx, Typ t, Ap ap, ApApprox apa, TypOption stored, RetNodeEx ret,
3139-
SummaryCtxSome innerSummaryCtx, ApApprox innerArgApa
3138+
SummaryCtxSome innerSummaryCtx
31403139
) {
31413140
fwdFlowThrough0(call, arg, cc, state, ccc, summaryCtx, t, ap, apa, stored, ret,
3142-
innerSummaryCtx, innerArgApa)
3141+
innerSummaryCtx, _)
31433142
}
31443143

31453144
bindingset[node, state, cc, summaryCtx, t, ap, stored]
@@ -3165,14 +3164,14 @@ module MakeImpl<LocationSig Location, InputSig<Location> Lang> {
31653164
private predicate fwdFlowThroughStep1(
31663165
PathNodeImpl pn1, PathNodeImpl pn2, PathNodeImpl pn3, DataFlowCall call, Cc cc,
31673166
FlowState state, CcCall ccc, SummaryCtx summaryCtx, Typ t, Ap ap, ApApprox apa,
3168-
TypOption stored, RetNodeEx ret, ApApprox innerArgApa
3167+
TypOption stored, RetNodeEx ret
31693168
) {
31703169
exists(
31713170
FlowState state0, ArgNodeEx arg, SummaryCtxSome innerSummaryCtx, ParamNodeEx p,
31723171
Typ innerArgT, Ap innerArgAp, TypOption innerArgStored
31733172
|
31743173
fwdFlowThroughStep0(call, arg, cc, state, ccc, summaryCtx, t, ap, apa, stored, ret,
3175-
innerSummaryCtx, innerArgApa) and
3174+
innerSummaryCtx) and
31763175
innerSummaryCtx = TSummaryCtxSome(p, state0, innerArgT, innerArgAp, innerArgStored) and
31773176
pn1 = mkPathNode(arg, state0, cc, summaryCtx, innerArgT, innerArgAp, innerArgStored) and
31783177
pn2 =
@@ -3189,7 +3188,7 @@ module MakeImpl<LocationSig Location, InputSig<Location> Lang> {
31893188
) {
31903189
exists(DataFlowCall call, CcCall ccc, RetNodeEx ret, boolean allowsFieldFlow |
31913190
fwdFlowThroughStep1(pn1, pn2, pn3, call, cc, state, ccc, summaryCtx, t, ap, _, stored,
3192-
ret, _) and
3191+
ret) and
31933192
flowThroughOutOfCall(call, ccc, ret, node, allowsFieldFlow) and
31943193
not inBarrier(node, state) and
31953194
if allowsFieldFlow = false then ap instanceof ApNil else any()

0 commit comments

Comments
 (0)