Skip to content

Fix and guard against marker zero-length string showing random data#1728

Merged
nullsystem merged 1 commit intoNeotokyoRebuild:masterfrom
nullsystem:FixZeroLenMarkerStr
Feb 24, 2026
Merged

Fix and guard against marker zero-length string showing random data#1728
nullsystem merged 1 commit intoNeotokyoRebuild:masterfrom
nullsystem:FixZeroLenMarkerStr

Conversation

@nullsystem
Copy link
Collaborator

@nullsystem nullsystem commented Feb 23, 2026

Description

The string wasn't zero-initalized, max length set 0 means it copies nothing and then it shows random data. Only an issue on Windows it seems but nevertheless need to have more checks to guard against this. Don't need to print text anyway if it's zero-length.

Toolchain

  • Linux GCC Distro Native Arch/GCC 15 (Although this issue not affecting Linux)

The string wasn't zero-initalized, max length set 0 means it copies
nothing and then it shows random data. Only an issue on Windows it
seems but nevertheless need to have more checks to guard against
this.
@nullsystem nullsystem requested a review from a team February 23, 2026 23:06
@Rainyan Rainyan requested a review from a team February 24, 2026 06:34
@kassibuss kassibuss linked an issue Feb 24, 2026 that may be closed by this pull request
2 tasks
@nullsystem nullsystem merged commit ee2378d into NeotokyoRebuild:master Feb 24, 2026
7 checks passed
@DESTROYGIRL DESTROYGIRL added the UI/HUD Relates to the HUD, NeoUI, menus, etc label Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI/HUD Relates to the HUD, NeoUI, menus, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Setting max name length to zero causes player names to turn to gibberish

4 participants