Submission Details
Severity:
Proxy contracts should not have a constructor()
Vulnerability Details
https://github.com/Cyfrin/2024-08-tadle/blob/main/src/core/TokenManager.sol#L28
Proxied contract can't have a constructor and it's common to move constructor logic to an external initializer function, usually called initialize
Impact
This issue could cause problems in the Storage of the contract.
Tools Used
Manual Review.
Recommendations
Use initializer functions for initialization of proxied contracts. All logic regarding initialization should be in the initialize function.
Prize pool breakdown
Total prize
30,000 USDC
nSLOC
1,22924 USDC / LOC
25,000 USDC
2,750 USDC
2,250 USDC
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.