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

Buyers funds will be stuck in buyOutEstateNFT function

Vulnerability Details

After transferring of funds from the buyer to the contract, we distribute the nft value to beneficiaries. Buyer is also a beneficiary so we skip him during payments. Problem here is instead of using continue we are using return keyword, which causes function to return early without executing the remaining code.

Impact

Funds will be stuck in the contract due to use return statment.

Tools Used

Manual Review

Recommendations

Replace return with continue

Updates

Lead Judging Commences

0xtimefliez Lead Judge 5 months ago
Submission Judgement Published
Validated
Assigned finding tags:

buyOutNFT has return instead of continue

Support

FAQs

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