Description

In the claimThrone() function, the game promises that the previous King should get a small cut of the new claim fee (as stated in the comment):

// If there's a previous king, a small portion of the new claim fee is sent to them.

However, in the actual code:

This silently violates the core game rule and causes the previous King to receive nothing, even though they were supposed to earn a reward for holding the throne.

This not only makes the game unfair but also damages user trust, and may cause players to stop participating once they realize the system is rigged or broken

Proof of Concept

Recommended Mitigation

Add a percentage value that defines how much of the claim fee should go to the previous king (e.g., previousKingCutPercentage).
Then, calculate the previous king’s payout from the amount sent by the new player (sentAmount), and store it safely like this