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

tr2/shell: fix crash when exiting with -l #2603

Merged
merged 1 commit into from
Mar 11, 2025
Merged

tr2/shell: fix crash when exiting with -l #2603

merged 1 commit into from
Mar 11, 2025

Conversation

rr-
Copy link
Collaborator

@rr- rr- commented Mar 11, 2025

Checklist

  • I have read the coding conventions
  • I have added a changelog entry about what my pull request accomplishes, or it is an internal change
  • I have added a readme entry about my new feature or OG bug fix, or it is a different change

Description

Fixes a crash when the player exits the game ran with -l / --level argument. I mistakenly believed that the level paths use a shared area allocation due to a leftover const TR2-specific specialization of the char *path GF_LEVEL member, but it's just Memory_DupStr()'d in both engines. Fixed the confusing define.

@rr- rr- added TRX bug A bug with TRX TR2 labels Mar 11, 2025
@rr- rr- self-assigned this Mar 11, 2025
@rr- rr- requested review from a team as code owners March 11, 2025 19:14
@rr- rr- requested review from lahm86, walkawayy and aredfan and removed request for a team March 11, 2025 19:14
@rr- rr- added the Unreleased Applies to things that aren't in a stable release. label Mar 11, 2025
@rr- rr- merged commit 92837a0 into develop Mar 11, 2025
9 checks passed
@rr- rr- deleted the l-arg branch March 11, 2025 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TR2 TRX bug A bug with TRX Unreleased Applies to things that aren't in a stable release.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants