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

[Crash]: Downloading a Game Script while having another Game Script selected may crash #10274

Closed
James103 opened this issue Dec 22, 2022 · 2 comments
Labels
bug Something isn't working component: AI/Game script (squirrel) This issue is related to Squirrel (Scripting language)

Comments

@James103
Copy link
Contributor

Version of OpenTTD

a857ed8 on Windows 10, 64-bit (via Steam)

Steps to reproduce

  1. Download some Game Scripts, including one that has parameters.
  2. From the main menu, click "Game Script Settings".
  3. Select an installed Game Script with parameters.
  4. From the console, run rescangame. The game may or may not crash right then.
  5. Click "Check Online Content".
  6. Download some more Game Scripts, then close the content downloading window which will run rescangame again to detect the newly downloaded Game Scripts.
  7. The game may or may not crash then as well. If not, repeat from step 3 with a different Game Script selected.

Upload crash files

crash20221222153328.zip
crash20221222153102.zip

Originally reported in JGRennison/OpenTTD-patches#465, but I was able to reproduce in vanilla.

@glx22
Copy link
Contributor

glx22 commented Dec 22, 2022

Seems to be a nasty one, crash.log are mentioning infinite loops.

@JGRennison
Copy link
Contributor

JGRennison commented Dec 22, 2022

It's just a dangling pointer heap use after free type bug. I'll sort something out later if I get time.

Edit: Better stack traces here: JGRennison/OpenTTD-patches#465 (comment)

@2TallTyler 2TallTyler added bug Something isn't working component: AI/Game script (squirrel) This issue is related to Squirrel (Scripting language) labels Dec 22, 2022
JGRennison added a commit to JGRennison/OpenTTD-patches that referenced this issue Dec 22, 2022
JGRennison added a commit to JGRennison/Upstream-OpenTTD that referenced this issue Dec 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working component: AI/Game script (squirrel) This issue is related to Squirrel (Scripting language)
Projects
None yet
Development

No branches or pull requests

4 participants