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

No Access Control on fund_pizza_drop

Root + Impact

Description

  • fund_pizza_drop can be called by anyone if owner check bypassed.

coin::transfer<AptosCoin>(owner, resource_addr, amount); // @> called without proper ownership check

Risk

Likelihood:

  • Non-owner funds can be sent or maliciously misused.

Impact:

  • Contract receives unauthorized funds.

  • Could disrupt accounting.

Proof of Concept

// Any user calls fund_pizza_drop with arbitrary amount

Recommended Mitigation

+ assert!(signer::address_of(owner) == state.owner, E_NOT_OWNER);
- coin::transfer<AptosCoin>(owner, resource_addr, amount);
+ coin::transfer<AptosCoin>(owner, resource_addr, amount);
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.