DeFiFoundry
60,000 USDC
View results
Submission Details
Severity: low
Invalid

" forwarder" can be address(0).

Summary

There is no check for zero address check for forwardwe.

Vulnerability Details

function setForwarder(address forwarder) external onlyOwner {
BaseKeeperStorage storage self = _getBaseKeeperStorage();
@> self.forwarder = forwarder;
}

Impact

we will have a zero forwader address.

Tools Used

Recommendations

require(forwader!=address(0));

Updates

Lead Judging Commences

inallhonesty Lead Judge
about 1 year ago
inallhonesty Lead Judge about 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.