Skip to content

fix(util): prevent non-tar copy path traversal on release-25#4802

Open
brendandburns wants to merge 2 commits into
release-25from
copilot/cherry-pick-4798-release-25
Open

fix(util): prevent non-tar copy path traversal on release-25#4802
brendandburns wants to merge 2 commits into
release-25from
copilot/cherry-pick-4798-release-25

Conversation

@brendandburns

Copy link
Copy Markdown
Contributor

Harden Copy non-tar directory copy against path traversal from pod-controlled ls -F output by sanitizing entry names and enforcing destination path containment. Add regression test for traversal and positive test for safe entries.

Harden Copy non-tar directory copy against path traversal from pod-controlled ls -F output by sanitizing entry names and enforcing destination path containment. Add regression test for traversal and positive test for safe entries.
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: brendandburns

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot requested a review from yue9944882 June 8, 2026 21:20
@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jun 8, 2026
@brendandburns

Copy link
Copy Markdown
Contributor Author

@copilot please fix this PR by updating all of the github actions to use SHA.

@linux-foundation-easycla

Copy link
Copy Markdown

CLA Not Signed

@k8s-ci-robot k8s-ci-robot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. and removed cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jun 8, 2026

Copilot AI commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

@copilot please fix this PR by updating all of the github actions to use SHA.

Done. All GitHub Actions in the workflow files are now pinned to immutable commit SHAs (with the version tag preserved as a comment), committed in 5a2ef36.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants