Submission Details
Severity:
Missing zero check conditional
Summary
In FjordStaking.sol there's a function FjordStaking.sol::addAuthorizedSablierSender which hasn't any conditional check to check whether address is a zero address or not.
Vulnerability Details
function addAuthorizedSablierSender(address _address) external onlyOwner {
@> // missing zero address check
authorizedSablierSenders[_address] = true;
} }
Impact
There may have zero address registered as an authorized sablier sender.
Tools Used
Manual Review.
Recommendations
Please implement a zero address check like below...
function addAuthorizedSablierSender(address _address) external onlyOwner {
+ if (_address == address(0)) revert InvalidZeroAddress();
authorizedSablierSenders[_address] = true;
}
Prize pool breakdown
Total prize
20,000 USDC
nSLOC
66230 USDC / LOC
16,000 USDC
2,500 USDC
1,500 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.