Skip to content
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.

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

Vulnerability resolutions #261

Closed
wants to merge 3 commits into from
Closed

Conversation

Zwuiix-cmd
Copy link

No description provided.

@Zwuiix-cmd
Copy link
Author

Let me explain a little: if the LoginPacket actually sends the chain, but with lots of erroneous data, you can simply freeze the server.

Then about the emoteId, with a little research I realized that if you sent it with a big enough size, i.e. a huge string, the client didn't like it at all and would freeze for a while.

As for the PurchaseReceiptPacket, I simply check that the data is valid, which I obtained by debugging on TheHive.

@dktapps dktapps closed this Aug 14, 2024
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.

2 participants