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

Ignore anonymous userdata folder #126

Merged
merged 1 commit into from
Sep 2, 2022
Merged

Ignore anonymous userdata folder #126

merged 1 commit into from
Sep 2, 2022

Conversation

DareFox
Copy link
Contributor

@DareFox DareFox commented Aug 25, 2022

The app crashes when trying to read anonymous user config file, 'cause it doesn't have required PersonaName field

Loading overlays...
Loaded 6 overlays. 

You can find many user-created overlays at https://www.reddit.com/r/steamgrid/wiki/overlays .

Looking for Steam directory...
If SteamGrid doesn´t find the directory automatically, launch it with an argument linking to the Steam directory.
Loading users...
Setting permission...
Setting permission...
Setting permission...
panic: runtime error: index out of range [1] with length 0

goroutine 1 [running]:
main.GetUsers({0x9c1aa60, 0x1a})
        C:/Users/BoppreH/Desktop/source/steamgrid/users.go:68 +0x66f
main.startApplication()
        C:/Users/BoppreH/Desktop/source/steamgrid/steamgrid.go:114 +0x13ff
main.main()
        C:/Users/BoppreH/Desktop/source/steamgrid/steamgrid.go:27 +0x44

Also, it doesn't make sense to fetch grids for anonymous user, since the only purpose for it is to download CLI game servers.

This PR adds a check to exclude anonymous folder from user parsing, and it should fix crash issue

@boppreh boppreh merged commit 36f069e into boppreh:master Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants