Bug 1953848 (CVE-2019-10743) - CVE-2019-10743 mholt/archiver: aribtrary file write via unsanitized destination filepaths
Summary: CVE-2019-10743 mholt/archiver: aribtrary file write via unsanitized destinati...
Keywords:
Status: NEW
Alias: CVE-2019-10743
Product: Security Response
Classification: Other
Component: vulnerability
Version: unspecified
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1953852
TreeView+ depends on / blocked
 
Reported: 2021-04-27 05:54 UTC by Sam Fowler
Modified: 2023-10-25 17:21 UTC (History)
7 users (show)

Fixed In Version: mholt/archiver 3.3.2
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Embargoed:


Attachments (Terms of Use)

Description Sam Fowler 2021-04-27 05:54:40 UTC
All versions of archiver allow attacker to perform a Zip Slip attack via the "unarchive" functions. It is exploited using a specially crafted zip archive, that holds path traversal filenames. When exploited, a filename in a malicious archive is concatenated to the target extraction directory, which results in the final path ending up outside of the target folder. For instance, a zip may hold a file with a "../../file.exe" location and thus break out of the target folder. If an executable or a configuration file is overwritten with a file containing malicious code, the problem can turn into an arbitrary code execution issue quite easily. 


References:

https://github.com/mholt/archiver/pull/169
https://snyk.io/research/zip-slip-vulnerability
https://snyk.io/vuln/SNYK-GOLANG-GITHUBCOMMHOLTARCHIVERCMDARC-174728

Comment 2 Mark Cooper 2021-04-30 00:32:20 UTC
Whilst the snyk report says to update the version of github.com/mholt/archiver/cmd/arc to v3.3.2 

Looking at the patch:
https://github.com/mholt/archiver/commit/8217ed3a206c0473b4ec1aff51375b398838073a

Its the mhold/archiver/ library in general, as it looks like calling archiver.go:Unarchive can lead to the vulnerable code path. Also the patch itself doesn't apply to cmd/ but to the more general package files.

Comment 3 Mark Cooper 2021-04-30 00:34:24 UTC
External References:

https://snyk.io/vuln/SNYK-GOLANG-GITHUBCOMMHOLTARCHIVERCMDARC-174728


Note You need to log in before you can comment on or make changes to this bug.