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

Emitting Event after Ownership Transfer

Summary

The setOwner function should emit an event when the ownership is transferred. This helps in tracking changes and improves transparency.

https://github.com/Cyfrin/2024-08-fjord/blob/main/src/FjordAuctionFactory.sol#L41

Recommendations

Emit an OwnerSet event after the owner is changed.

event OwnerSet(address indexed newOwner);
function setOwner(address _newOwner) external onlyOwner {
if (_newOwner == address(0)) revert InvalidAddress();
owner = _newOwner;
emit OwnerSet(_newOwner);
}
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.