manager.sol
Last updated
Last updated
Description
There is no guarantee that the owner deposits will satisfy the presale and liquidity rates.
Description
masterCopy
is unused and unnecessary. In addition It will be overridden by the implementation contract, making its usage dangerous.
Recommendation
Remove this variable.
Description
The proxy contract call can be frontrun by a malicious user who deploys to the same address as the legitimate user because the salt is generated only by the address and timestamp.
Recommendation
Use a more secure salt that cannot cause denial of service.
Description
Proxy contracts are very vulnerable and may easily cause high severity logical and security issues by misusing them.
Recommendation
Make sure you are familiar with the potential risks of using proxy contracts and have handled them properly.
Description
A struct can be used to pass multiple variables in a more descriptive and readable way.
Description
There are many different places in the code containing "magic numbers". There is no way for us to know what they mean and therefore, review the logic of the code containing them. In addition, parts containing "magic numbers" are more error prone.
For example:
Recommendation
Change these values into variables or constants with descriptive names.
Description
If these values must be equal to a constant you know beforehand, why are they received externally? Just use the const.
Type
Severity
Location
Status
Logical Issue
High
initializePresale
Resolved
Type
Severity
Location
Status
Best Practice
Medium
Proxy
Resolved
Type
Severity
Location
Status
Best Practice
Medium
createNewLaunchpad, createNewFairLaunch
Resolved
Type
Severity
Location
Status
Best Practice
Medium
createNewLaunchpad, createNewFairLaunch
Acknowledged
Type
Severity
Location
Status
Best Practice
Low
createNewLaunchpad, createNewFairLaunch
Not Resolved
Type
Severity
Location
Status
Best Practice
Medium
global
Resolved
Type
Severity
Location
Status
Best Practice
Informational
createNewLaunchpad, createNewFairLaunch
Not Resolved