PHI Token Smart Contract audit report.
This is the report from a security audit performed on PHI Token by MrCrambo.
The audit focused primarily on the security of PHI Token Smart Contract.
In total, 4 issue were reported including:
-
0 high severity issues.
-
2 medium severity issues.
-
2 low severity issues.
According to documentation there is 60000 tokens for sale, but in code there is 60250
Please provide correct data and re-check all the values.
Using setTransferActive
owner can disable transfer for all people not from whitelist, but transfers should be active after ICO end.
Add condition, that if ICO ended, then all the transfers are enabled.
mint
function should create new tokens, but in this case it will just transfer from other address.It's same with transferFrom
function.
- It is possible to double withdrawal attack. More details here
- Lack of transaction handling mechanism issue. More details here
Smart contract has two medium severity issues, please fix it before deploying.