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]: Crash when changing settings when config file is synced using Dropbox #10474
Comments
|
I'm not able to reproduce, and I don't understand the call flow. |
Same problem with Here's screenrecord of triggering the crash. Here's my openttd.cfg in case it's something to do with one of the setting. |
Looking at the crash log and your comments.
|
|
Did another test with Windows 10, OpenTTD from official build (ZIP version), same crash, same repro.
|
So you get the same crash with JGRPP. Can you tell which release started to show the issue ? (as JGRPP is updated more often than vanilla it may give a hint). |
With JGR 0.50.0, I constantly got a crash after repeated change the value 5 times (Just double-click the 'Effect of distance on demand' and change to any values multiple times. With JGR 0.50.1 and later, I constantly got a crash after 1 or 2 times at most. On the other hand, I did some more investigation and I was now absolutely baffled. So I download a ZIP file of 13.0 from the website, and I create blank openttd.cfg to make it not read my main config. No crash. I copied all my .cfg (openttd, window, hotkeys, private, secret) file from my Document folder to the aforementioned folder. No crash. ["Portable config"] I renamed openttd.cfg to openttd2.cfg (so make it read the system setting again). Crash. ["System config"] It seems that the even with the "Portable config" it still read basesets and GRF from the system folder? So both config should be technically the same, but it has different result? Am I missing something here? |
Okay, I figured it out. So my Documents\openttd folder is actually symlink'd to Dropbox (so all my config and GRF is backed up and sync to all my machine). Dropbox is probably locking the This is probably recent Dropbox change because I never have this problem before, and blaming the git history show that this part of code hasn't been changed in 2 years. I will leave this open if you think this is a valid problem, or you can close it if you think it's outside the scope. |
Not that I have the solution, but might it be DropBox having a "copy hook" in the file copying/renaming operation which for some reason triggers a re-entrant call to |
Might this be the same issue as #12477? If so, it might have been solved in the nightly and the next bugfix release (14.1). |
Version of OpenTTD
13.0, Windows 11
Steps to reproduce
(A lot of other activities related to game setting also crash at the same spot)
For the provided save game, I changed the value from 30 to 20.
Upload crash files
crash.zip
The text was updated successfully, but these errors were encountered: