Skip to content

stdlib: Properly sanatize filenames when (un)zipping #9941

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

Merged
merged 2 commits into from
Jun 16, 2025

Conversation

garazdawi
Copy link
Contributor

No description provided.

garazdawi added 2 commits May 28, 2025 13:05
According to the Zip APPNOTE filenames "MUST NOT contain a drive or
device letter, or a leading slash.". So we strip those when zipping
and unzipping.
…s/stdlib/zip-sanitize-filename/27/OTP-19653

* lukas/stdlib/zip-sanitize-filename/OTP-19653:
  stdlib: Properly sanatize filenames when (un)zipping
@garazdawi garazdawi added this to the OTP-29.0 milestone Jun 13, 2025
@garazdawi garazdawi self-assigned this Jun 13, 2025
Copy link
Contributor

github-actions bot commented Jun 13, 2025

CT Test Results

  1 files   11 suites   3m 26s ⏱️
 95 tests  91 ✅ 4 💤 0 ❌
111 runs  107 ✅ 4 💤 0 ❌

Results for commit 62531c3.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@garazdawi garazdawi force-pushed the lukas/master/OTP-19653 branch from 62531c3 to b49c0b4 Compare June 16, 2025 08:19
@garazdawi garazdawi changed the title gh: Remove default labels and assignees from renovate prs stdlib: Properly sanatize filenames when (un)zipping Jun 16, 2025
@garazdawi garazdawi merged commit b0cca71 into erlang:master Jun 16, 2025
1 check passed
@garazdawi
Copy link
Contributor Author

Fixed in OTP-26.2.5.13, OTP-27.3.4.1 and OTP-28.0.1.

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.

1 participant