fix(game/five): verify pickup weapon component is valid on creation

[slashkeyvalue]

Goal of this PR

Prevent crash caused by invalid pickup weapon components on pickup creation.

How is this PR achieving the goal

Adds proper validations on pickup creation

This PR applies to the following area(s)

FiveM

Successfully tested on

Game builds: 2060, 3095

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

Fixes GTA5_b3095.exe!sub_141163854 (0x456)

[slashkeyvalue]

Is there something else pending or ...? Would be nice to be less abused by cheats :P

[Pedro-Lucas14]

Is the cheater taking down every player that some fix date goes into production? @prikolium-cfx

[TkzDev]

The cheater is taking down all players, do you have a fix date for it to go into production? @prikolium-cfx

[slashkeyvalue]

In case testing code is desired;

static uintptr_t g_writerVtbl;

static void (*origCPickupCreationDataNode__Serialise)(void* self, void* serialiser);
static void CPickupCreationDataNode__Serialise(void* self, void* serialiser)
{
	bool isWriter = *(uintptr_t*)serialiser == g_writerVtbl;

	if (isWriter)
	{
		*(uint32_t*)((uintptr_t)self + 0x128) = 0xFFFFFFFF; 
		*(uint32_t*)((uintptr_t)self + 0x158) = 0x0000000C;
	}

	origCPickupCreationDataNode__Serialise(self, serialiser);
}

static HookFunction hookFunction([]
{
	g_writerVtbl = hook::get_address<uintptr_t>(hook::get_pattern("4C 8B 10 48 8B C8 41 FF 52 ? 48 8B 4B 08", 0x11));

	origCPickupCreationDataNode__Serialise = hook::trampoline(hook::get_pattern("40 55 53 56 57 41 54 41 56 41 57 48 8B EC 48 83 EC ? 48 8B 02 4C 8D B1"), &CPickupCreationDataNode__Serialise);
});