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

Check if the second soulmate is existed or not in `getDivorced()::Soulmate.sol` contract.

Summary

There is no check for the soulmate 2 existence in the getDivorced() function.

Impact

If there is no soulmate 2 existed, couples cannot be divorced. And we cannot emit the event as CoupleHasDivorced inside the getDivorced() function.

Tools Used

Manually

Recommendations

Add require statement inside the getDivorced() function in Soulmate.sol contract:

+ require(soulmate2 != address(0), "Soulmate does not exist");
Updates

Lead Judging Commences

0xnevi Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Other

Support

FAQs

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