Skip to content
Snippets Groups Projects

Minor changes to prevent SIGSEGVs

Closed 256nil requested to merge 256nil/SRB2:sigsegv-inspection into next

Replace strncpy with memcpy to prevent conditional jump or move depending on uninitialised value(s). Done by using valgrind, @ [m_menu.c:2668]

LUA: Add checks to show a warning if the object being deleted in MobjSpawn hook is the one being spawned (to prevent the game from crashing). The checks are being done within the functions lib_pRemoveMobj and lib_pKillMobj, and the variable that is being used to detect the deletion of the spawning object is a boolean called inside_spawn, and is switched to true at the beginning of P_SpawnMobj. It is turned to false before the object is returned.

Add a check for objects with the flag MF_STICKY touching the sky to prevent them from being deleted if they're a player object. Because otherwise this crashes the game.

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
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading