Part 2

Zaros
PerpetualsDEXFoundrySolidity
70,000 USDC
View results
Submission Details
Severity: medium
Valid

MarketMakingEngineConfigurationBranch : the initialize function is not added. The same happen in other branch contract as well.

Summary

The contract is not implemented with initialize function which makes it not possible to set the owner address

Vulnerability Details

MarketMakingEngineConfigurationBranch is up-gradable contract branch contract. There are other branches implemented to work as individual entity and support the zaro market implementation.

when deploying the contract, the initialize of the each contract's function is planned to call. But these branch contract does not implement the initialize functions.

Impact

Branch contracts can not be initialized.

For example, the MarketMakingEngineConfigurationBranch contrct inherits owneable_upgradable. The contract can not set its owner.

Tools Used

Manual review.

Recommendations

Suggestion : add initialize function in each of the branches

Updates

Lead Judging Commences

inallhonesty Lead Judge 5 months ago
Submission Judgement Published
Validated
Assigned finding tags:

`_Ownable_init()` is not called in `MarketMakingEngineConfigurationBranch` contract

Support

FAQs

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