stake.link

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

Missing checks for address 0 in initializer

The initializer lacks proper checks for address zero, which may pose security risks and lead to unexpected behavior in the system.

Impact

The absence of checks for address zero in the initializer could potentially result in security vulnerabilities or unexpected behavior, compromising the integrity of the system.

Recommendation

It is crucial to implement thorough validation for address zero in the initializer to mitigate potential security risks and ensure the stability and security of the system.

+error InvalidAddress();
function __SDLPoolBase_init(
string memory _name,
string memory _symbol,
address _sdlToken,
address _boostController
) public onlyInitializing {
+ if (_sdlToken == address(0) || _boostController == address(0)) {
+ revert InvalidAddress();
+ }
__RewardsPoolController_init();
name = _name;
symbol = _symbol;
sdlToken = IERC20Upgradeable(_sdlToken);
boostController = IBoostController(_boostController);
}
Updates

Lead Judging Commences

0kage Lead Judge over 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.