TempleGold
TempleDAO
Foundry
25,000 USDC
View results
Previous Next
Submission Details
Severity: medium
Invalid

TempleGoldAdmin Lacks transferOwnership Implementation

0xCiphky

Vulnerability Details:

The TempleGoldAdmin contract is intended to be the owner of the TempleGold contract once it’s deployed and ownership is transferred. Therefore, the TempleGoldAdmin contract implements all onlyOwner functionalities from the TempleGold contract to make changes as needed.

However, the transferOwnership function is not implemented by TempleGoldAdmin, meaning if the ownership needs to be transferred, it currently cannot be done.

function transferOwnership(address newOwner) public virtual onlyOwner {
if (newOwner == address(0)) {
revert OwnableInvalidOwner(address(0));
}
_transferOwnership(newOwner);
}

Impact:

Ownership of the TempleGold contract cannot be transferred at all because the TempleGoldAdmin contract, which is the owner, does not implement the transferOwnership function.

Tools Used:

Manual analysis

Recommendation:

Implement the transferOwnership function in the TempleGoldAdmin contract to ensure that ownership can be transferred if needed.

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.