Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Bugfix: Fix NRE in DefaultLauncherAccountParser.cs #114

Merged
merged 1 commit into from
Jan 26, 2024
Merged

Bugfix: Fix NRE in DefaultLauncherAccountParser.cs #114

merged 1 commit into from
Jan 26, 2024

Conversation

laolarou726
Copy link
Member

@laolarou726 laolarou726 commented Jan 26, 2024 •
edited by pr-codex bot
Loading

PR-Codex overview

This PR focuses on adding a check for empty or null key before removing an account from the LauncherAccount's Accounts collection.

Detailed summary

  • Added a check for empty or null key before removing an account from LauncherAccount's Accounts collection.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@laolarou726 laolarou726 requested a review from BiDuang January 26, 2024 02:02
@laolarou726 laolarou726 added the 漏洞 Bug 某些功能发生了异常 Something isn't working label Jan 26, 2024
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@laolarou726 laolarou726 merged commit 4a1422d into master Jan 26, 2024
7 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@BiDuang BiDuang BiDuang approved these changes

Assignees

@laolarou726 laolarou726

@BiDuang BiDuang

Labels
漏洞 Bug 某些功能发生了异常 Something isn't working
Projects
ProjBobcat
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@laolarou726 @BiDuang