Submission Details
Severity:
Staking.sol - deposit() - Staking will fail using USDT on mainnet - use safeTransferFrom()
Summary
IERC20 OZ's interface of transferFrom() expects that a boolean is returned. That's not the case in USDT's implementation in Ethereum Mainnet. Such, the call TKN.transferFrom() when TKN is USDT will fail, blocking the contract functionality.
Vulnerability Details
Every call to deposit() will fail.
Impact
The contract will be useless
Tools Used
Manual Review
Recommendations
Use safeTransferFrom() instead.
Prize pool breakdown
Total prize
20,000 USDC
nSLOC
70628 USDC / LOC
18,000 USDC
2,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.