Beginner FriendlyFoundryNFT
100 EXP
View results
Submission Details
Severity: medium
Valid

Avoid using loop

Summary

Avoid using loop as it can reach the gas limit.

Vulnerability Details

If there is too much users, the transaction can revert when reaching the gas limit, and so the smart contract is not usable anymore...

Impact

Tools Used

Recommendations

Use a mapping, or an EnumerableMap from OpenZeppelin.
It can also simplify the getActivePlayerIndex function.

Updates

Lead Judging Commences

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

denial-of-service-in-enter-raffle

Support

FAQs

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