Skip to content

Fix jail item loop exploit - #125

Merged
GhzGarage merged 1 commit into
mainfrom
ghzgarage-fix-savejailitems-givejailitems-loop
Aug 26, 2026
Merged

GhzGarage merged 1 commit into
mainfrom
ghzgarage-fix-savejailitems-givejailitems-loop

Conversation

@GhzGarage

Copy link
Copy Markdown
Contributor

Block SaveJailItems and GiveJailItems unless the player is currently jailed (injail > 0), preventing remote event abuse for infinite jail cash.

Summary

Related issue

Change type

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Refactor, maintenance, or performance improvement

Testing

  • FXServer artifact:
  • qb-core version or commit:
  • Resource version or commit:
  • Server operating system:
  • Test steps and results:

Compatibility and migration

Screenshots or video

Checklist

  • I agree to follow the QBCore FiveM Code of Conduct.
  • My pull request title follows Conventional Commits, such as fix(scope): ... or feat(scope): ....
  • This pull request contains one focused change and does not include unrelated formatting or refactoring.
  • I tested the change on a current FXServer artifact with the relevant official resources and dependencies.
  • Existing repository checks and linting pass.
  • I documented new behavior and identified every breaking or migration-related change.
  • I added or updated configuration examples, SQL migrations, and translations where applicable.
  • I removed credentials, webhook URLs, license keys, database data, player identifiers, and other private information.
  • I have the right to submit all included code and assets under the repository's license.
  • I understand and reviewed all submitted code, including any AI-assisted code, and accept responsibility for its correctness and licensing.

Block SaveJailItems and GiveJailItems unless the player is currently jailed (injail > 0), preventing remote event abuse for infinite jail cash.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@GhzGarage
GhzGarage merged commit 4ac6a67 into main Aug 26, 2026
2 checks passed
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