Snowman Merkle Airdrop

First Flight #42
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Impact: medium
Likelihood: medium
Invalid

No Safety Checks on collectFee() Withdrawals

Summary

Failure of either WETH or ETH transfer may leave the contract in inconsistent state or lock funds.

Description

WETH transfer uses raw .transfer(), lacks success verification.
ETH fallback uses low-level .call(), but still may fail.
No partial recovery or rollback if one succeeds and other fails.

POC

no POC

Impact

Loss of funds.
Partial transfers.
Stuck balances.

Severity

MEDIUM

Likelihood

MEDIUM

Recommendation

Use safeTransfer for WETH.
Handle ETH failure with proper revert or fallback logic.

Updates

Lead Judging Commences

yeahchibyke Lead Judge 5 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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