Skip to content

Fix #37 - #43

Open
itziakos wants to merge 9 commits into
mainfrom
fix-#37
Open

Fix #37#43
itziakos wants to merge 9 commits into
mainfrom
fix-#37

Conversation

@itziakos

@itziakos itziakos commented Jul 4, 2026

Copy link
Copy Markdown
Owner
  • Add code to check and sanitize symlinks
  • Add tests to check against zip-slip
  • Add tests to check against writing symlinks that point outside the target folder

fixes #37

itziakos added 9 commits June 13, 2026 11:33
- Use pathlib for the path constants
- Add ZIP_SLIP and ZIP_SLIP_WIN contacts
- Factor out a private method to sanitize the arcname (i.e. path)
- Add check to make sure that the final target path is not outside
  of the extract root
- Use the new sanitize method to check that link source path is
  valid and not outside the extract root
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check against symlinking refering to out-of-archive files

1 participant