Skip to content

Engine should no longer crash on Mac due to a corrupted save file#284

Merged
BellringerQuinn merged 1 commit intomasterfrom
166-corrupted-sav-engine-crash
Feb 14, 2025
Merged

Engine should no longer crash on Mac due to a corrupted save file#284
BellringerQuinn merged 1 commit intomasterfrom
166-corrupted-sav-engine-crash

Conversation

@colezemind
Copy link
Contributor

@colezemind colezemind commented Feb 12, 2025

Removed a Try Catch statement from Apple Bridge Decrypt that wouldn't have been allowed. Updated NativeApplyEncryptor.mm to better handle situations of null or empty NSString values sent into Decrypt

Docs Checklist

Please ensure you have addressed documentation updates if needed as part of this PR:

  • I have created a separate PR on the sequence docs repository for documentation updates: Link to docs PR
  • No documentation update is needed for this change.

…have been allowed. Updated NativeApplyEncryptor.mm to better handle situations of null or empty NSString values sent into Decrypt
@colezemind colezemind requested a review from a team February 12, 2025 13:51
@colezemind colezemind requested a review from a team as a code owner February 12, 2025 13:51
Copy link
Contributor

@BellringerQuinn BellringerQuinn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@BellringerQuinn BellringerQuinn merged commit 06a635b into master Feb 14, 2025
1 of 2 checks passed
@BellringerQuinn BellringerQuinn deleted the 166-corrupted-sav-engine-crash branch February 14, 2025 16:23
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