Skip to content

Commit 5130135

Browse files
author
Alvaro Muñoz
committed
fix(stepsExpression): allow steps from a composite action to communicate
1 parent 778d897 commit 5130135

File tree

1 file changed

+8
-1
lines changed
  • ql/lib/codeql/actions/ast/internal

1 file changed

+8
-1
lines changed

ql/lib/codeql/actions/ast/internal/Ast.qll

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -855,7 +855,14 @@ class StepsExpressionImpl extends SimpleReferenceExpressionImpl {
855855
override string getFieldName() { result = fieldName }
856856

857857
override AstNodeImpl getTarget() {
858-
this.getEnclosingJob() = result.getEnclosingJob() and
858+
(
859+
this.getEnclosingJob() = result.getEnclosingJob()
860+
or
861+
exists(CompositeActionImpl a |
862+
a.getAChildNode*() = this and
863+
a.getAChildNode*() = result
864+
)
865+
) and
859866
result.(StepImpl).getId() = stepId
860867
}
861868

0 commit comments

Comments
 (0)