Skip to content

fix(gta-core-five): task NMShot behaviour failure exploit#3864

Open
DaniGP17 wants to merge 1 commit intocitizenfx:masterfrom
DaniGP17:fix/task-nmshot-failure-exploit
Open

fix(gta-core-five): task NMShot behaviour failure exploit#3864
DaniGP17 wants to merge 1 commit intocitizenfx:masterfrom
DaniGP17:fix/task-nmshot-failure-exploit

Conversation

@DaniGP17
Copy link
Contributor

Goal of this PR

Fix a client crash that could be triggered when CTaskNMShot::BehaviourFailure dereferences a null PedIntelligence pointer.

How is this PR achieving the goal

The PR adds a hook that validates the PedIntelligence pointer before it is dereferenced. If the pointer is null, the code path that accesses m_queriableInterface is skipped.

This PR applies to the following area(s)

FiveM

Successfully tested on

Game builds: 2060, 3258
Platforms: Windows

Checklist

  • Code compiles and has been tested successfully.
  • Code explains itself well and/or is documented.
  • My commit message explains what the changes do and what they are for.
  • No extra compilation warnings are added by these changes.

Thanks @imLocutor for reporting this exploit

@github-actions github-actions bot added the invalid Requires changes before it's considered valid and can be (re)triaged label Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid Requires changes before it's considered valid and can be (re)triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant