MyCut

First Flight #23
Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: high
Valid

Wrong calculation in `Pot.sol` for `claimantCut`

Summary

In the Pot.solthe calculation for claimantCutis wrongly calculated.

Vulnerability Details

The claimantCutshould be calculated like that uint256 claimantCut = (remainingRewards - managerCut) / claimants.length;but instead it is calculated like this uint256 claimantCut = (remainingRewards - managerCut) / i_players.length;.

https://github.com/Cyfrin/2024-08-MyCut/blob/946231db0fe717039429a11706717be568d03b54/src/Pot.sol#L57

Impact

This can lead to wrong calculated cuts and unintented behaviour.

Tools Used

Manual review

Recommendations

Change i_players.length to claimants.length.

Updates

Lead Judging Commences

equious Lead Judge 11 months ago
Submission Judgement Published
Validated
Assigned finding tags:

Incorrect distribution in closePot()

Support

FAQs

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