stake.link

stake.link
DeFiHardhatBridge
27,500 USDC
View results
Submission Details
Severity: low
Invalid

Large approvals may not work with some `ERC20` tokens

Summary

Large approvals may not work with some ERC20 tokens

Vulnerability Details

Not all IERC20 implementations are totally compliant, and some (e.g UNI, COMP) may fail if the valued passed is larger than uint96. Source

Impact

Instances (5):

File: contracts/core/ccip/SDLPoolCCIPControllerPrimary.sol
193: IERC20(_tokens[i]).safeApprove(router, type(uint256).max);

193

File: contracts/core/ccip/WrappedTokenBridge.sol
71: linkToken.approve(_router, type(uint256).max);
72: token.approve(_wrappedToken, type(uint256).max);

71, 72

File: contracts/core/ccip/base/SDLPoolCCIPController.sol
52: linkToken.approve(_router, type(uint256).max);
53: sdlToken.approve(_router, type(uint256).max);

52, 53

Tools Used

Recommendations

Updates

Lead Judging Commences

0kage Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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