Skip to content

fix: pass vector arguments of NPC proxies by value - #611

Open
NeekoGta wants to merge 3 commits into
ikkentim:mainfrom
NeekoGta:main
Open

NeekoGta wants to merge 3 commits into
ikkentim:mainfrom
NeekoGta:main

Conversation

@NeekoGta

@NeekoGta NeekoGta commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Fix crash on Linux x64 when using NPC functions taking vectors.

The INPC and INPCComponent proxies took Vector3/GTAQuat parameters as const references, while the managed side passes them by value.

Pass the current player to `INetwork.GetStatistics` from the `Player` network properties.

These properties called `GetStatistics()` without a player, so the API returned network-wide statistics.
In particular, `ConnectionStatus` reported `NoAction` for connected players.
Fix crash on Linux x64 when using NPC functions taking vectors.

The INPC and INPCComponent proxies took Vector3/GTAQuat parameters as
const references, while the managed side passes them by value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant