Beginner FriendlyFoundryGameFi
100 EXP
View results
Submission Details
Severity: high
Valid

User can get infinite rewards (HealthToken).

Summary

User can get infinite HealthTokens by calling the updateCountMartenitsaTokensOwner function, as it has no access control.

Vulnerability Details

The updateCountMartenitsaTokensOwner function can be called multiple times adding countMartenitsaTokensOwner to himself, because there is no access control.
Than by calling collectReward the malicious user can get HealthTokens for every 3 requiredMartenitsaTokens.

Impact

High because a user can get infinite HealthTokens.

Tools Used

Manual review

Recommendations

Add access control to updateCountMartenitsaTokensOwner function.

Updates

Lead Judging Commences

bube Lead Judge over 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

Missing access control

Support

FAQs

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