Sablier
Sablier
DeFiFoundry
53,440 USDC
View results
Previous Next
Submission Details
Severity: medium
Invalid

ERC20 Tokens implementing Dividends will lead to loss of funds

tom2o17

Summary

Tokens with dividend payouts on transfer(address,uint256) will lead to loss funds

POC use as asset: eth:0xbed85cf4c249bd5fc187af600d652dd2beefddea as token

eg:
eth:0xde84a421726fe25ff5795e76de0b88af5eaddc7ed0c72f6b003466a29f8b852f

In addition, if a Token has an ETH/native gas dividend payout all transfers may revert

├─ [51] 0xB10daee1FCF62243aE27776D7a92D39dC8740f95::fallback{value: 100000000000000000}()
│ └─ ← "EvmError: Revert"
└─ ← ()

Mitigation:

Implement rescue function if both sender, admin && recipient agree to recover funds

Make contract have a payable fallback so if someone integrates a token which pays gas, does not revert on transfer.

Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Known issue

Support

FAQs

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