Does not check uniqueness of ShareHolder #231
Labels
2 (Med Risk)
Assets not at direct risk, but function/availability of the protocol could be impacted or leak value
bug
Something isn't working
duplicate
This issue or pull request already exists
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
Handle
hack3r-0m
Vulnerability details
https://github.com/code-423n4/2021-11-nested/blob/main/contracts/FeeSplitter.sol#L264
does not check if there is already a shareholder before creating a new shareholder.
this will cause an issue in
findShareHolder
since it will return the first shareholder in the array while there are more than one.The text was updated successfully, but these errors were encountered: