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

Fix ahnyQuab ogg decompression crashes. #1507

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

Hoikas
Copy link
Member

@Hoikas Hoikas commented Oct 17, 2023

This issue was introduced when hsStream::Close() was removed in favor of RAII in #1448. Unfortunately, the patcher is handing off files before it deletes the patcher stream, which causes race conditions around zlib decompression not being complete. We need to explicitly close the underlying stream before performing any handoffs to prevent crashes.

Fixes #1506

This issue was introduced when `hsStream::Close()` was removed in favor
of RAII. Unfortunately, the patcher is handing off files before it
deletes the patcher stream, which causes race conditions around zlib
decompression not being complete. We need to explicitly close the
underlying stream before performing any handoffs to prevent crashes.
@dpogue
Copy link
Member

dpogue commented Oct 17, 2023

Can confirm that this allowed me to patch the entire client data set from TrollLand with no issues.

@dpogue dpogue merged commit 5f460c8 into H-uru:master Oct 17, 2023
14 checks passed
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.

Patcher crashes on OGG files
3 participants