Skip to content
Snippets Groups Projects

Fix a few strncpy bullshit

Merged LJ Sonic requested to merge strncpy-brullshit into next

Turns out the codebase is filled with calls to strncpy on non null-terminated strings, which causes !885 (merged) to not work properly in some cases.

I tried fixing that but fixing all such strings is a lot of work, so I defaulted to reverting the old entry searching functions, and instead adding an alternate version that can find long names, which is used by Lua only for now.

Ideally the long-term goal would be to eventually completely get rid of the old functions, and use only the long name ones.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading