Skip to content

Commit 42fd9f0

Browse files
authored
Merge pull request github#14047 from hvitved/dataflow/join-fix
Data flow: Fix a bad join order
2 parents a7de0f9 + f2eed4d commit 42fd9f0

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

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

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1623,12 +1623,11 @@ module MakeImpl<InputSig Lang> {
16231623
DataFlowCall call, ArgNodeEx arg, ParamNodeEx p, boolean allowsFieldFlow, Ap argAp, Ap ap
16241624
) {
16251625
exists(ApApprox argApa, Typ argT |
1626-
flowIntoCallApa(call, _, pragma[only_bind_into](arg), pragma[only_bind_into](p),
1627-
allowsFieldFlow, argApa) and
1626+
returnFlowsThrough(_, _, _, _, pragma[only_bind_into](p), pragma[only_bind_into](argT),
1627+
pragma[only_bind_into](argAp), ap) and
1628+
flowIntoCallApa(call, _, pragma[only_bind_into](arg), p, allowsFieldFlow, argApa) and
16281629
fwdFlow(arg, _, _, _, _, _, pragma[only_bind_into](argT), pragma[only_bind_into](argAp),
16291630
argApa) and
1630-
returnFlowsThrough(_, _, _, _, p, pragma[only_bind_into](argT),
1631-
pragma[only_bind_into](argAp), ap) and
16321631
if allowsFieldFlow = false then argAp instanceof ApNil else any()
16331632
)
16341633
}

0 commit comments

Comments
 (0)