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

UI Error after being kicked by NSDisconnectPlayer during loading screen #849

Open
Bobbyperson opened this issue Aug 29, 2024 · 3 comments · May be fixed by #887
Open

UI Error after being kicked by NSDisconnectPlayer during loading screen #849

Bobbyperson opened this issue Aug 29, 2024 · 3 comments · May be fixed by #887
Assignees
Labels
bug Something isn't working

Comments

@Bobbyperson
Copy link
Contributor

image

[SCRIPT UI] [info] SCRIPT ERROR: [UI] Entity is null
[SCRIPT UI] [info]  -> int loadoutIndex = player.GetPersistentVarAsInt( loadoutType + "SpawnLoadout.index" )
[SCRIPT UI] [info] 
CALLSTACK
*FUNCTION [GetPersistentSpawnLoadoutIndex()] sh_loadouts.nut line [2815]
*FUNCTION [UpdateCachedLoadouts_Threaded()] sh_progression.nut line [172]

[SCRIPT UI] [info] LOCALS
[loadoutType] "pilot"
[player] NULL
[this] NULL
[this] NULL

DIAGPRINTS

Steps to reproduce:

  1. Get kicked by NSDisconnectPlayer while loading into a game
  2. Observe results

This was observed on prior Northstar versions (at least a few major versions back) and has been reproduced by me on latest.

@EM4Volts
Copy link
Contributor

This happens if you connect to a server / map before loading has finished for the main menu or the end of gsme challenge stuff has properly finished. If anyone wants to reproduce

@GeckoEidechse GeckoEidechse added the bug Something isn't working label Aug 29, 2024
@Jan200101
Copy link

Ran into this by coincidence
looks to be a race condition in UpdateCachedLoadouts_Threaded, it is trying to get the loadout after the player entity has already been nulled.

solution would be to check if the player entity is valid either in GetPersistentSpawnLoadoutIndex or UpdateCachedLoadouts_Threaded

@GeckoEidechse
Copy link
Member

cc @Zanieon

@Zanieon Zanieon self-assigned this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: No status
Status: Zanieon TODOs
Development

Successfully merging a pull request may close this issue.

5 participants