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

CVE-2007-4559 Patch #7892

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

TrellixVulnTeam
Copy link

Patching CVE-2007-4559

Hi, we are security researchers from the Advanced Research Center at Trellix. We have began a campaign to patch a widespread bug named CVE-2007-4559. CVE-2007-4559 is a 15 year old bug in the Python tarfile package. By using extract() or extractall() on a tarfile object without sanitizing input, a maliciously crafted .tar file could perform a directory path traversal attack. We found at least one unsantized extractall() in your codebase and are providing a patch for you via pull request. The patch essentially checks to see if all tarfile members will be extracted safely and throws an exception otherwise. We encourage you to use this patch or your own solution to secure against CVE-2007-4559. Further technical information about the vulnerability can be found in this blog.

If you have further questions you may contact us through this projects lead researcher Kasimir Schulz.

@d2iq-mergebot
Copy link
Collaborator

This repo has @mesosphere-mergebot integration. You can perform the following commands by submitting a comment. Submit a comment with content "@mesosphere-mergebot help" to view more detailed help text and examples. Be sure the have a look at the mergebot documentation, too.

@mesosphere-mergebot bump-ee  
@mesosphere-mergebot override-status pr-status-check jira-url 
@mesosphere-mergebot sync  
@mesosphere-mergebot label [dependencies|Ship It|Ready For Review|Request For Comment|do not merge - test|merge-by-mergebot|Holding|Work In Progress] 
@mesosphere-mergebot autotest-on-backports-and-trains  
@mesosphere-mergebot changelog-not-required reason 
@mesosphere-mergebot backport  
@mesosphere-mergebot merge-it  
@mesosphere-mergebot ready-for-review-label  
@mesosphere-mergebot test  
@mesosphere-mergebot request-review  
@mesosphere-mergebot autotrigger-tests-on-ready-for-review  
  • PR creators can apply one of [Ready For Review|Work In Progress]. Owners can apply any label.

@d2iq-mergebot
Copy link
Collaborator

@mesosphere-mergebot bump-ee

@d2iq-mergebot
Copy link
Collaborator

Enterprise Bump PR: mesosphere/dcos-enterprise/pull/8552

@cgawron1
Copy link
Collaborator

cgawron1 commented Nov 7, 2022

@mesosphere-mergebot test

@d2iq-mergebot
Copy link
Collaborator

Expected Format: @mesosphere-mergebot test check_1 check_2 ...

@cgawron1
Copy link
Collaborator

cgawron1 commented Nov 7, 2022

@mesosphere-mergebot test all

@d2iq-mergebot
Copy link
Collaborator

Successfully triggered checks:

teamcity/dcos/build/tox
teamcity/dcos/build/dcos
teamcity/dcos/test/terraform/aws/onprem/static/group4
teamcity/dcos/test/test-e2e/group1
teamcity/dcos/test/terraform/aws/onprem/static/group1
teamcity/dcos/test/terraform/aws/onprem/static/group2
teamcity/dcos/test/terraform/aws/onprem/static/group3

@cgawron1
Copy link
Collaborator

cgawron1 commented Nov 7, 2022

@TrellixVulnTeam you have a few issues that need addressing. let us start with getting continuous-integration/jenkins/pr-head to pass. there are also some trailing whitespaces that need removing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants