Omniscia 0xPhase Audit

OwnableBase Manual Review Findings

OwnableBase Manual Review Findings

OBE-01M: Inexistent Initialization of Ownership

TypeSeverityLocation
Logical FaultOwnableBase.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.