[86fe8c] openrct2.exe: Guard::Assert_VA, Guard::Assert, window_tile_inspector_get_selected_element, window_tile_inspector_inv...

[OpenRCT2-git-bot]

New error: [openrct2] 1 occurrence since (Fri Nov 15 2019 11:27:39)

Application	openrct2.exe
Attributes	dummy — 1 0 — 1 attachment_replay.sv6r — 1 C:\Users\mbz49\OneDrive\Documents\OpenRCT2\7afa1e1a-1e3a-405c-ac72-feb1d1b561f9.sv6r — 1 attachment_park.sv6 — 1 * — 1 attachment_config.ini — 1 C:\Users\mbz49\OneDrive\Documents\OpenRCT2\7afa1e1a-1e3a-405c-ac72-feb1d1b561f9.ini — 1 assert_failure — 1 Selected list item out of range — 1 attachment_park.sv6.gz — 1 C:\Users\mbz49\OneDrive\Documents\OpenRCT2\7afa1e1a-1e3a-405c-ac72-feb1d1b561f9.sv6.gz — 1 attachment_screenshot.png — 1 C:\Users\mbz49\OneDrive\Documents\OpenRCT2\screenshot\Cape Royal Comet 2021-02-19 09-31-52.png — 1 commit — 1 e11304f — 1
Classifiers	invalid-write
Frames	Guard::Assert_VA Guard::Assert window_tile_inspector_get_selected_element window_tile_inspector_invalidate window_find_widget_from_point GameHandleInputMouse GameHandleInput context_handle_input OpenRCT2::GameState::Update OpenRCT2::Context::Update OpenRCT2::Context::RunVariableFrame OpenRCT2::Context::RunGameLoop OpenRCT2::Context::RunOpenRCT2 NormalisedMain wmain

[Gymnasiast]

I guess this assert could get hit if two people are working on the same tile in multiplayer. In which case, we should put this in a comment and just return a nullptr.

@Broxzier would you agree?

[Broxzier]

@Gymnasiast Looking at the screenshot it looks like the player was not in multiplayer, but in construction. It might be that they had a ghost track element selected that no longer exists.

Not sure if this is this is right, but the selected tile also has a ton of track elements at the same height on it. This might have something to do with the crash.

[github-actions]

This issue is stale and will be closed tomorrow if no action is taken. To keep it open, leave a comment or remove the stale-backtrace label.

[Gymnasiast]

Please keep it open for a bit.

[github-actions]

This issue is stale and will be closed tomorrow if no action is taken. To keep it open, leave a comment or remove the stale-backtrace label.

[Gymnasiast]

@Broxzier What do you think we should do with this?

[Broxzier]

I suspect there is more happening here than we can see, considering there are a ton of elements on that tile. With no steps to reproduce this, and the backtrace report gone, I don't think keeping this open will help much. I have not seen any reports like this since.