Skip to content

tweak(game/five): expand support for weapon info entries #3412

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

tweak(game/five): expand support for weapon info entries #3412

wants to merge 1 commit into from

Conversation

Effectss
Copy link

@Effectss Effectss commented May 21, 2025 •
edited
Loading

Goal of this PR

Prevent server crashes caused by exceeding the default CWeaponInfoBlob limit.

How is this PR achieving the goal

By increasing the CWeaponInfoBlob allocation limit from 512 to 1024, this PR ensures stability for servers with a large number of custom or modified weapons. Without this adjustment, servers crash when the default limit is surpassed.

This PR applies to the following area(s)

This change impacts the FiveM runtime environment.

Successfully tested on

Wasn't tested on anything

Checklist

  • Code compiles and has been tested successfully.
  • Code explains itself well and/or is documented.
  • [ x ] 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 weapon limits to fix crashing issue once the limit is passed

@github-actions github-actions bot added the invalid Requires changes before it's considered valid and can be (re)triaged label May 21, 2025
@Effectss Effectss changed the title increase CWeaponInfoBlob limit fix(fxd):increase CWeaponInfoBlob limit May 21, 2025
@Effectss Effectss changed the title fix(fxd):increase CWeaponInfoBlob limit fix(fxd): increase CWeaponInfoBlob limit May 21, 2025
@Effectss Effectss changed the title fix(fxd): increase CWeaponInfoBlob limit fix(core/five): increase CWeaponInfoBlob limit May 21, 2025
@Effectss Effectss changed the title fix(core/five): increase CWeaponInfoBlob limit fix(local.h): increase CWeaponInfoBlob limit May 21, 2025
@Effectss Effectss changed the title fix(local.h): increase CWeaponInfoBlob limit tweak(game/five): increase CWeaponInfoBlob limit May 21, 2025
@Effectss
Copy link
Author

Value increase is to help fix weapon crashes when going over this limit

@Effectss Effectss changed the title tweak(game/five): increase CWeaponInfoBlob limit tweak(game/five): expand support for weapon info entries May 21, 2025
@github-actions github-actions bot added triage Needs a preliminary assessment to determine the urgency and required action and removed invalid Requires changes before it's considered valid and can be (re)triaged labels May 21, 2025
@Effectss
Copy link
Author

Any updates?

@CronusWaxxed
Copy link

This would be amazing because I am having an issue adding guns into my server

@github-actions github-actions bot added invalid Requires changes before it's considered valid and can be (re)triaged and removed triage Needs a preliminary assessment to determine the urgency and required action labels May 23, 2025
@Effectss Effectss changed the title tweak(game/five): expand support for weapon info entries tweak(game/five): expand support for weapon info entrie Jun 2, 2025
@Effectss Effectss changed the title tweak(game/five): expand support for weapon info entrie tweak(game/five): expand support for weapon info entries Jun 2, 2025
@Effectss Effectss closed this Jun 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
invalid Requires changes before it's considered valid and can be (re)triaged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Effectss @CronusWaxxed