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

Missing Registration Check in claim_pizza_slice

Root + Impact

Description

  • claim_pizza_slice uses users_claimed_amount without confirming registration properly.

let amount = *table::borrow(&state.users_claimed_amount, user_addr); // @> may panic if not registered

Risk

Likelihood:

  • Unregistered users may trigger panic.

Impact:

  • Transaction failure, potential DoS.

Proof of Concept

// Call claim_pizza_slice without prior registration

Recommended Mitigation

+ assert!(table::contains(&state.users_claimed_amount, user_addr), E_NOT_REGISTERED);
- let amount = *table::borrow(&state.users_claimed_amount, user_addr);
+ let amount = *table::borrow(&state.users_claimed_amount, user_addr);
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.