Early rate$2,400 of senior audit time for $500. Early members keep the rate as it climbs.$2,400 of senior audit time for $500See how →
F-2024-0003·input-validation

Decimals are not required in the deposit function

Acknowledgedpentestblackboxcoin-flip
TL;DR

Decimal handling on deposit was unconstrained, allowing values that the backend may not handle cleanly and creating drift between the displayed amount and the on-chain settlement.

Severity
MEDIUM
Impact
MEDIUM
Likelihood
MEDIUM
Method
MManual review
CAT.
Complexity
LOW
Exploitability
MEDIUM
02Section · Description

Description

The deposit function did not require or enforce a specific decimal format. Users could enter decimal values that the application accepted at the UI layer, but the backend was not necessarily designed to handle the resulting partial values cleanly. This mirrors the decimal-rounding class observed across the Virtual Rollups game audits (All Your Base, Beecasino, Dedprz).

03Section · Impact

Impact

Display drift between the UI's decimal representation and any backend numeric handling. Potential gas griefing or settlement failures if the on-chain layer rejects decimal-tail values.

04Section · Recommendation

Recommendation

Decide whether decimals are required for deposits. If not, reject them at the input boundary; if so, normalize to a fixed-precision representation that the backend explicitly supports, and use that canonical numeric format end-to-end.

Hopium: Acknowledged. Zealynx: Acknowledged.

F-2024-0003