Skip to content

Corruption fixes maybe? - #270

Merged
donkeyProgramming merged 1 commit into
donkeyProgramming:masterfrom
Pear-231:CorruptionFixesMaybe
Dec 5, 2025
Merged

donkeyProgramming merged 1 commit into
donkeyProgramming:masterfrom
Pear-231:CorruptionFixesMaybe

Conversation

@Pear-231

Copy link
Copy Markdown
Collaborator
  • Fixed a potential filename length calculation issue due to some special characters by using UTF-8 byte count instead of character count.
  • Fixed potential issue with temp files by using FileMode.Create instead to prevent reusing incorrect data.
  • Moved SystemFilePath to after file move to prevent reads from a potentially wrong file.
  • Added assertion against saving with encryption as we don't support that.

- Fixed a potential filename length calculation issue due to some special characters by using UTF-8 byte count instead of character count.
- Fixed potential issue with temp files by using FileMode.Create instead to prevent reusing incorrect data.
- Moved SystemFilePath to after file move to prevent reads from a potentially wrong file.
- Added assertion against saving with encryption as we don't support that.
@donkeyProgramming
donkeyProgramming merged commit a86a0ef into donkeyProgramming:master Dec 5, 2025
1 check passed
@Pear-231
Pear-231 deleted the CorruptionFixesMaybe branch December 7, 2025 22:27
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.

2 participants