DatingDapp

First Flight #33
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Severity: low
Invalid

[M-1] `LikeRegistry.sol` receive function doesn't apply functionality, leading to stuck funds

Summary

The receive function in LikeRegistry.sol only receives funds but does nothing with them:

receive() external payable {}

Impact

Though low likelyhood, if a user calls LikeRegistry.sol and sends funds to increase the ETH from his/her previous like, those funds will be lost forever, since they don't get considered in any functionality.

Recommendations

Consider making LikeRegistry::likeUser a public function so that it can be called within the receive.

Updates

Appeal created

n0kto Lead Judge 7 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

invalid_receive_function

Not the best design, but if you send money accidentally, that's a user mistake. Informational.

Support

FAQs

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