stake.link

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

Missing Zero Address Validation

The setCCIPController function in the SDLPool contract sets the ccipController address without validating that the provided address is not the zero address. Assigning the zero address could lead to unexpected behavior or loss of control over certain functionalities.

Problematic Code:

function setCCIPController(address _ccipController) external onlyOwner {
ccipController = _ccipController;
}

To prevent potential issues, add a check to ensure that _ccipController is not the zero address before setting it.

function setCCIPController(address _ccipController) external onlyOwner {
require(_ccipController != address(0), "CCIPController cannot be the zero address");
ccipController = _ccipController;
}
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.