Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-51777] Don't let tar entries escape target dir #3482

Merged
merged 1 commit into from
Jun 9, 2018

Conversation

daniel-beck
Copy link
Member

See JENKINS-51777.

Basically complete the half of https://snyk.io/research/zip-slip-vulnerability in core that #3402 left out. We don't currently consider this to be exploitable, so this is hardening, and we had to hold it back due to the embargo until this week.

Proposed changelog entries

  • Security hardening: Prevent tar archives from escaping their base directory.

Submitter checklist

  • JIRA issue is well described
  • Changelog entry appropriate for the audience affected by the change (users or developer, depending on the change). Examples
    * Use the Internal: prefix if the change has no user-visible impact (API, test frameworks, etc.)
  • Appropriate autotests or explanation to why this change has no tests
  • [n/a] For dependency updates: links to external changelogs and, if possible, full diffs

Desired reviewers

@Wadeck

@daniel-beck daniel-beck requested a review from Wadeck June 8, 2018 10:21
@Wadeck
Copy link
Contributor

Wadeck commented Jun 8, 2018

Perhaps a test case could be interesting

@oleg-nenashev oleg-nenashev added the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label Jun 8, 2018
@oleg-nenashev
Copy link
Member

Test failure is unrelated

@oleg-nenashev oleg-nenashev merged commit 1afd9f8 into jenkinsci:master Jun 9, 2018
@JESUSANSWERS777
Copy link

JESUSANSWERS777 commented Jun 9, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants