Trick or Treat

First Flight #27
Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: low
Invalid

Redundant changeOwner Function

Summary

The Ownable contract already provides transferOwnership, making changeOwner redundant

Vulnerability Detail

function changeOwner(address _newOwner) public onlyOwner {
transferOwnership(_newOwner);
}

Impact

Redundant code can lead to increased maintenance overhead and potential security risks if not carefully managed.

Tools Used

Manual review

Recommendations

Remove changeOwner to rely solely on transferOwnership, which is part of OpenZeppelin’s tested and secure Ownable implementation.

Updates

Appeal created

bube Lead Judge 8 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.