DatingDapp

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

Permanently Locked ETH in Contract

  • Root Cause: The contract lacks a mechanism to withdraw ETH sent by users (except via withdrawFees, which only handles fees). Since userBalances are not updated, all ETH remains stuck.

  • Impact: Users cannot recover their ETH, leading to permanent fund loss.

  • Recommendation:

    • Add an emergency withdrawal function (guarded by multi-sig or timelock) to recover stuck ETH.

    • Fix the userBalances tracking issue as above to prevent future locking.

Updates

Appeal created

n0kto Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Design choice
Assigned finding tags:

invalid_no_withdrawing_function_and_like_all_used

Money collected will be sent to the MultisigWallet during the first match. Emergency withdraw could lead to a frontrun before a match. "If the like is mutual, all their previous like payments (minus a 10% fee) are pooled into a shared multisig wallet" Design choice

Support

FAQs

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