Omniscia 0xPhase Audit
OwnableBase Manual Review Findings
OwnableBase Manual Review Findings
OBE-01M: Inexistent Initialization of Ownership
| Type | Severity | Location |
|---|---|---|
| Logical Fault | ![]() | OwnableBase.sol:L24-L33 |
Description:
The OwnableBase contract does not contain any method to initialize its first ownership similarly to other facets of the system, such as ERC20Base::_initializeERC20.
Example:
diamond/Ownable/OwnableBase.sol
6abstract contract OwnableBase {Recommendation:
We advise the code to introduce an initialization mechanism meant to be invoked during the facet's registration to ensure that ownership is properly retained in the Diamond instance.
Alleviation (3dd3d7bf0c2693b2f9c23bacedfa420393f7ea84):
The code of OwnableBase was updated to expose a new Ownable::_initializeOwnable function that performs the correct initialization steps of the contract akin to the rest of the codebase. As such, we consider this exhibit fully alleviated.
