Missing _disableInitializers(); in KeeperProxy
Summary:
The KeeperProxy contract is an Upgradeable Contract, so the Openzeppelin's function _disableInitializers(); needs to be called in its constructor.
Vulnerability Details:
If the _disableInitializers(); function is not called in its constructor, an attacker could take over the ownership of this contract by re-initializing the contract.
Tools Used:
Manual review
Recommendations:
To prevent this you should invoke the _disableInitializers(); function from Openzeppelin.
Lead Judging Commences
Informational or Gas
Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelihood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.
Prize pool breakdown
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.