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

FAF client freezing and crashing when switching between tables and tiles in rapid succession #2954

Closed
magge-faf opened this issue Apr 16, 2023 · 2 comments · Fixed by #2962
Closed
Labels

Comments

@magge-faf
Copy link
Collaborator

When switching between tables and tiles in a short timeframe (<1s), the FAF client freezes and crashes. The error can be reproduced consistently by switching between tables and tiles 6-12 times in rapid succession. It is possible that this error occurs with regular tabs as well when the client has been open for a longer period.

Steps to reproduce:

Open the FAF client
Switch between tables and tiles 6-12 times in rapid succession (<1s)
The FAF client freezes and crashes

"Unexpected exception from an event executor" error that was previously reported. The error message in the Client.log file indicates a Java heap space error.
client.log

image

Expected behavior:

The FAF client should not freeze or crash when switching between tables and tiles.

Actual behavior:

The FAF client freezes and crashes when switching between tables and tiles in rapid succession.

Additional information:

The fix of this issue solves maybe this one as well #2952

@magge-faf magge-faf added the bug label Apr 16, 2023
@MantisMaestro
Copy link

Had a bit of a play around, there definitely seems to be a memory leak in the CustomGamesController, regarding the GamesTilesContainerController and the GamesTableController. I've not dealt with Java for a long while though so I could be barking up the wrong tree. I'll take a better look tonight and see what else I can find.

@magge-faf
Copy link
Collaborator Author

Thanks for the fast fix - Amazing work as usual. The CPU spikes when switching tabs disappeared, and the whole UI feels more responsive now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants