Skip to content

GUI: Use AsciiString instead of const char* for nextScreen in WOL menus - #92

Open
seer-by-sentry[bot] wants to merge 1 commit into
mainfrom
seer/ascii-string-nextscreen
Open

GUI: Use AsciiString instead of const char* for nextScreen in WOL menus#92
seer-by-sentry[bot] wants to merge 1 commit into
mainfrom
seer/ascii-string-nextscreen

Conversation

@seer-by-sentry

Copy link
Copy Markdown

Fixes CLIENT-KQ. The issue was that: Access violation during std::basic_string construction caused by hash table rehash attempting to read corrupted string data during WOLQuickMatchMenuInit.

  • Replaced const char* nextScreen with AsciiString nextScreen in WOLWelcomeMenu.cpp and WOLQuickMatchMenu.cpp.
  • Modified the shutdown logic to use nextScreen.isNotEmpty() instead of nextScreen != NULL.
  • Used nextScreen.clear() instead of nextScreen = NULL to clear the string.

This fix was generated by Seer in Sentry, triggered automatically. 👁️ Run ID: 2073043

Not quite right? Click here to continue debugging with Seer.

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.

0 participants