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

Adopt the two-step ownership transfer pattern in `FjordAuctionFactory`, `FjordStaking` and `FjordPoints`

Vulnerability Details

The function setOwner() transfers ownership to a new address. In case a wrong address is supplied, ownership is inaccessible and the factory is unusable.

FjordAuctionFactory.sol#L41-L44

function setOwner(address _newOwner) external onlyOwner {
if (_newOwner == address(0)) revert InvalidAddress();
owner = _newOwner;
}

Recommendations

Adopt the two-step ownership transfer pattern:

  1. the owner sets the new owner

  2. the new owner accepts the ownership

Updates

Lead Judging Commences

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.