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

NFTs and ETH Can Be Bruned

Summary

NFTs and ETH can be burned if player calls refund

Vulnerability Details

When a player calls refund the address at their index in the players array is set to address(0). If the winner index happens to be address(0) the winning funds and NFT will be burned and lost forever

Impact

Loss of NFTs and ETH

Tools Used

Manual Review

Recommendations

When deleting something from an array it is best to shift the index you want to delete to the end of the array and use .pop to remove it.

Updates

Lead Judging Commences

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

zero address can win the raffle

Funds are locked to no one. If someone gets the refund issue, they also got this issue. IMPACT: High Likelihood: High

Support

FAQs

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