feat(gta-core-five): clamp vehicle entrypoint index

[divocbn]

Goal of this PR

Prevent a crash caused by cheaters sending invalid vehicle entry/exit point indices that are serialized and synchronized in cloned vehicle tasks.

How is this PR achieving the goal

This PR clamps the serialized target entry/exit point index for CTaskEnterVehicle and CTaskExitVehicle to the valid range defined by the vehicle model entry/exit points.

This PR applies to the following area(s)

FiveM

Successfully tested on

Game builds: 3258, 3570, 3717

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.

Fixes issues

[st860923]

Should there be a guard for numEntryExitPoints == 0 before the clamp?
The game itself seem early-outs when the count is zero.

[divocbn]

Should there be a guard for numEntryExitPoints == 0 before the clamp? The game itself seem early-outs when the count is zero.

Kinda sure that it should be right as is, especially because 2-seat vehicles has 0 numEntryExitPoints.
Also my proof-of-concept crasher isn't working after that fix, could you provide any screens

[st860923]

You're right. I traced through the downstream code and the function already guards against numEntryExitPoints == 0 by skipping the lookup loop entirely and returning false, so the value is never actually used. The fix works as intended. 👍

[radium-cfx]

Looks good.

Thank you!