Beginner FriendlyGameFi
100 EXP
View results
Submission Details
Impact: high
Likelihood: medium
Invalid

Double Claim Vulnerability

Root + Impact

Description

  • Users can call claim_pizza_slice multiple times if claimed_users updated after transfer.

table::add(&mut state.claimed_users, user_addr, true); // @> after transfer

Risk

Likelihood:

  • Attacker calls claim repeatedly before state updated.

Impact:

  • Same user claims multiple slices, draining pool.

Proof of Concept

// Call claim_pizza_slice twice quickly, receive double payout

Recommended Mitigation

- table::add(&mut state.claimed_users, user_addr, true);
+ table::add(&mut state.claimed_users, user_addr, true); // move before transfer
Updates

Appeal created

bube Lead Judge 12 days ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.