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 in window_sign_invalidate due to original bug #4455

Closed
1 task done
remcooo96 opened this issue Sep 19, 2016 · 7 comments · Fixed by #5969
Closed
1 task done

Crash in window_sign_invalidate due to original bug #4455

remcooo96 opened this issue Sep 19, 2016 · 7 comments · Fixed by #5969
Labels
crash Game exits suddenly.

Comments

@remcooo96
Copy link

remcooo96 commented Sep 19, 2016

Mac OSX
0.0.5
426e106

  • Reproducible in RCT2 (vanilla)? YES, tested

I have signs in my park that cannot be deleted. Al these "ghost signs" are connected, if I want to give one of them a new name, whatever sign I choose it always shows the same location in the little window (the place it shows doesn't even have a sign. When I change its name, all the names of the ghost signs adopt this name. In openrct even touching the signs lead to a crash, in vanilla just the above described happens.

I will add the dump file from macos and the savegame (some ghost signs are in the middle of the screen when you open it)

crash report.zip
Astoria10000.sv6.zip

@Gymnasiast Gymnasiast added investigate Not sure what the problem is yet. crash Game exits suddenly. labels Sep 19, 2016
@janisozaur
Copy link
Member

janisozaur commented Sep 19, 2016

0   website.openrct2.OpenRCT2       0x020bed83 window_sign_invalidate + 54
1   website.openrct2.OpenRCT2       0x0208f598 window_draw + 518
2   website.openrct2.OpenRCT2       0x020d677c window_draw_all + 215
3   website.openrct2.OpenRCT2       0x0207740e SoftwareDrawingEngine::DrawDirtyBlocks(unsigned int, unsigned int, unsigned int, unsigned int) + 224
4   website.openrct2.OpenRCT2       0x02076f2c SoftwareDrawingEngine::DrawAllDirtyBlocks() + 170
5   website.openrct2.OpenRCT2       0x02076845 SoftwareDrawingEngine::Draw() + 131
6   website.openrct2.OpenRCT2       0x0210079b drawing_engine_draw + 26
7   website.openrct2.OpenRCT2       0x02054faf openrct2_launch + 972
8   website.openrct2.OpenRCT2       0x0204a1fa main + 41
9   website.openrct2.OpenRCT2       0x020011f5 start + 53
Thread 0 crashed with X86 Thread State (32-bit):
  eax: 0x00000000  ebx: 0x00000000  ecx: 0x00000000  edx: 0x0229a03c
  edi: 0x022f4cdc  esi: 0x020bed5a  ebp: 0xbffffab8  esp: 0xbffffaa0
   ss: 0x00000023  efl: 0x00210246  eip: 0x020bed83   cs: 0x0000001b
   ds: 0x00000023   es: 0x00000023   fs: 0x00000000   gs: 0x0000000f
  cr2: 0x00000007

scenery_entry must be NULL:

rct_scenery_entry* scenery_entry = get_large_scenery_entry(w->var_48C);

@remcooo96
Copy link
Author

Okay thats probably the problem, but can you explain what it means. Im no programming wonder, sorry ;)

@janisozaur janisozaur removed the investigate Not sure what the problem is yet. label Sep 19, 2016
@janisozaur
Copy link
Member

It means we know what the problem is, bringing us one step closer to solution. Making it not crash would probably be simple, but fixing such signs properly (so that you can actually delete it or change its value) may require some more effort.

@duncanspumpkin
Copy link
Contributor

How did you create the ghost signs. There was a bug in vanilla that causes them but I fixed that for open

@remcooo96
Copy link
Author

I have honestly no idea, I just started noticing them when I tried to delete one, which caused the game to crash. I know the roots of my savegame are made in vanilla and were then build further in Openrct.

@duncanspumpkin
Copy link
Contributor

The bug that caused them in vanilla was when building two signs with same direction on different paths that share x y coordinates but different heights.

@remcooo96
Copy link
Author

I think I indeed had two signs at different levels. I hope it is fixable. At least stopping the game from crashing would be big improvement :)

@janisozaur janisozaur changed the title Ghost signs, crash Crash in window_sign_invalidate due to original bug Nov 4, 2016
IntelOrca pushed a commit that referenced this issue Jul 22, 2017
janisozaur added a commit to janisozaur/OpenRCT2 that referenced this issue Aug 9, 2017
- Feature: [OpenRCT2#5815] Add cheat to ignore research status and access rides/scenery not yet invented.
- Feature: [OpenRCT2#5857] Keyboard shortcut for clear scenery.
- Feature: [OpenRCT2#5877] Allow up to 16 stations to be synchronised
- Feature: [OpenRCT2#5970] The Bobsleigh Roller Coaster now supports on-ride photos.
- Feature: [OpenRCT2#5991] Allow all tracked rides that can be tested without guests to the Track Designer
- Fix: [OpenRCT2#2127, OpenRCT2#2229, OpenRCT2#5586] Mountain tool cost calculation
- Fix: [OpenRCT2#3589] Crash due to invalid footpathEntry in path_paint
- Fix: [OpenRCT2#3852] Constructing path not clearing scenery on server.
- Fix: [OpenRCT2#4455] Crash in window_sign_invalidate due to original bug
- Fix: [OpenRCT2#4715] Fix OpenGL rendering of water when zoomed. See OpenRCT2#5890.
- Fix: [OpenRCT2#4931] Crash in path_paint - footpathentry was null
- Fix: [OpenRCT2#5629] Issue with tower ride modes approach to station - incorrect sum caused sawtooth in velocity
- Fix: [OpenRCT2#5768] Prevent loading non-existent title sequences.
- Fix: [OpenRCT2#5858] Crash when using custom ride with no colour presets.
- Fix: [OpenRCT2#5865] Ride preview flickering on uneven terrain or mid air.
- Fix: [OpenRCT2#5872] Incorrect OpenGL rendering of masked sprites
- Fix: [OpenRCT2#5880] Leaving bumper cars without building causes assertion.
- Fix: [OpenRCT2#5890] Fix zoomed OpenGL rendering of special sprites with primary and secondary colours.
- Fix: [OpenRCT2#5912] Negative queue when moving entrance in paused state.
- Fix: [OpenRCT2#5920] Placing guest spawn doesn't do anything every 3rd click
- Fix: [OpenRCT2#5939] Crash when importing 'Six Flags Santa Fe'.
- Fix: [OpenRCT2#5977] Custom music files not showing up in music list
- Fix: [OpenRCT2#5981] Ride list doesn't update after using quick demolish.
- Fix: [OpenRCT2#5984] Allow socket binding to same port after crash
- Fix: [OpenRCT2#5998] Staff not getting paid / no loan interest.
- Fix: [OpenRCT2#6026] 'Select ride to advertise' dropdown does not display all items.
- Fix: [OpenRCT2#6052] Unable to place entrance/exit on certain ride types.
- Fix: [OpenRCT2#6071] Quick demolish can delete protected ride.
- Fix: [OpenRCT2#6111] Mute button always visible in editor.
- Fix: [OpenRCT2#6113] Track preview shows incorrect highest drop height.
- Improved: [OpenRCT2#2223] Change mountain tool to ignore higher surrounding tiles.
- Improved: [OpenRCT2#4301] Leading and trailing whitespace in player name is now removed.
- Improved: [OpenRCT2#5859] OpenGL rendering performance
- Improved: [OpenRCT2#5863] Switching drawing engines no longer requires the application to restart.
- Improved: [OpenRCT2#6003] Doors placed on tracks will now work with all vehicles
- Improved: [OpenRCT2#6037] Autosaves are now stored in a subfolder
- Improved: The land tool buttons can now be held down to increase/decrease size.
- Improved: Dropdowns longer than 32 items overflow into columns.
- Improved: Ride Type option in ride window is now a dropdown.
- Improved: "About OpenRCT2" window redesigned, now contains OpenRCT2 info and access to changelog
janisozaur added a commit that referenced this issue Aug 9, 2017
- Feature: [#5815] Add cheat to ignore research status and access rides/scenery not yet invented.
- Feature: [#5857] Keyboard shortcut for clear scenery.
- Feature: [#5877] Allow up to 16 stations to be synchronised
- Feature: [#5970] The Bobsleigh Roller Coaster now supports on-ride photos.
- Feature: [#5991] Allow all tracked rides that can be tested without guests to the Track Designer
- Fix: [#2127, #2229, #5586] Mountain tool cost calculation
- Fix: [#3589] Crash due to invalid footpathEntry in path_paint
- Fix: [#3852] Constructing path not clearing scenery on server.
- Fix: [#4455] Crash in window_sign_invalidate due to original bug
- Fix: [#4715] Fix OpenGL rendering of water when zoomed. See #5890.
- Fix: [#4931] Crash in path_paint - footpathentry was null
- Fix: [#5629] Issue with tower ride modes approach to station - incorrect sum caused sawtooth in velocity
- Fix: [#5768] Prevent loading non-existent title sequences.
- Fix: [#5858] Crash when using custom ride with no colour presets.
- Fix: [#5865] Ride preview flickering on uneven terrain or mid air.
- Fix: [#5872] Incorrect OpenGL rendering of masked sprites
- Fix: [#5880] Leaving bumper cars without building causes assertion.
- Fix: [#5890] Fix zoomed OpenGL rendering of special sprites with primary and secondary colours.
- Fix: [#5912] Negative queue when moving entrance in paused state.
- Fix: [#5920] Placing guest spawn doesn't do anything every 3rd click
- Fix: [#5939] Crash when importing 'Six Flags Santa Fe'.
- Fix: [#5977] Custom music files not showing up in music list
- Fix: [#5981] Ride list doesn't update after using quick demolish.
- Fix: [#5984] Allow socket binding to same port after crash
- Fix: [#5998] Staff not getting paid / no loan interest.
- Fix: [#6026] 'Select ride to advertise' dropdown does not display all items.
- Fix: [#6052] Unable to place entrance/exit on certain ride types.
- Fix: [#6071] Quick demolish can delete protected ride.
- Fix: [#6111] Mute button always visible in editor.
- Fix: [#6113] Track preview shows incorrect highest drop height.
- Improved: [#2223] Change mountain tool to ignore higher surrounding tiles.
- Improved: [#4301] Leading and trailing whitespace in player name is now removed.
- Improved: [#5859] OpenGL rendering performance
- Improved: [#5863] Switching drawing engines no longer requires the application to restart.
- Improved: [#6003] Doors placed on tracks will now work with all vehicles
- Improved: [#6037] Autosaves are now stored in a subfolder
- Improved: The land tool buttons can now be held down to increase/decrease size.
- Improved: Dropdowns longer than 32 items overflow into columns.
- Improved: Ride Type option in ride window is now a dropdown.
- Improved: "About OpenRCT2" window redesigned, now contains OpenRCT2 info and access to changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crash Game exits suddenly.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants