Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Improve verify #200

Merged
merged 2 commits into from
Jan 6, 2025
Merged

Improve verify #200

merged 2 commits into from
Jan 6, 2025

Conversation

bdon
Copy link
Member

@bdon bdon commented Jan 6, 2025

  • properly exit with status != 0 when invalid archive detected
  • detect missing magic number instead of crashing
  • detect mismatch between header data length and actual file length
  • detect bounds area <= 0

bdon added 2 commits January 6, 2025 12:32
@bdon
Improve verify
34454ab 
* properly exit with status != 0 when invalid archive detected
* detect missing magic number instead of crashing
* detect mismatch between header data length and actual file length
* detect bounds area <= 0
@bdon
formatting
9a1abad
@bdon bdon merged commit 967ba9d into main Jan 6, 2025
4 checks passed
@bdon bdon deleted the stricter-verify branch January 6, 2025 05:00
bdon added a commit that referenced this pull request Jan 6, 2025
@bdon
make verify robust for padded archives [#200]
1860585
bdon added a commit that referenced this pull request Jan 6, 2025
@bdon
make verify robust for padded archives [#200] (#201)
cb7d438
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bdon