Omniscia Mitosis Audit

BasicVaultFactory Static Analysis Findings

BasicVaultFactory Static Analysis Findings

BVF-01S: Inexistent Event Emission

Description:

The linked function adjusts a sensitive contract variable yet does not emit an event for it.

Example:

src/vault/BasicVaultFactory.sol
132function updateHub(address hub) external onlyOwner {
133 _getStorageV1()._hub = hub;
134}

Recommendation:

We advise an event to be declared and correspondingly emitted to ensure off-chain processes can properly react to this system adjustment.

Alleviation (58e8cc66dfa900c03c47df78f5170d9960005629):

The HubUpdated event was introduced to the codebase and is correspondingly emitted in the BasicVaultFactory::updateHub function, addressing this exhibit in full.