Skip to content

Commit c11fac8

Browse files
committed
Make summaryThroughStepValue include param outputs
This matches summaryThroughStepTaint.
1 parent 8198b1a commit c11fac8

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

shared/dataflow/codeql/dataflow/internal/FlowSummaryImpl.qll

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1286,10 +1286,8 @@ module Make<
12861286
* be useful to include in the exposed local data-flow/taint-tracking relations.
12871287
*/
12881288
predicate summaryThroughStepValue(ArgNode arg, Node out, SummarizedCallable sc) {
1289-
exists(ReturnKind rk, SummaryNode ret, DataFlowCall call |
1290-
summaryLocalStep(summaryArgParam(call, arg, sc), ret, true, _) and
1291-
summaryReturnNode(ret, pragma[only_bind_into](rk)) and
1292-
out = getAnOutNode(call, pragma[only_bind_into](rk))
1289+
exists(SummaryNode ret |
1290+
summaryLocalStep(summaryArgParamRetOut(arg, ret, out, sc), ret, true, _)
12931291
)
12941292
}
12951293

0 commit comments

Comments
 (0)