This document is a security audit report performed by danbogd, where VANM token has been reviewed.
- VANMToken.sol github commit hash 1c0995c827b200c76e90efe3a809096a71251581.
In total,3 issues were reported including:
- 2 low severity issues.
- 1 minor observation
No critical security issues were found.
-
It is possible to double withdrawal attack. More details here.
-
Lack of transaction handling mechanism issue. WARNING! This is a very common issue and it already caused millions of dollars losses for lots of token users! More details here.
Extra checking in 279, 294-295 lines. SafeMath library checks it anyway.
https://github.com/VANMofficial/Token-Sale/blob/1c0995c827b200c76e90efe3a809096a71251581/VANMToken.sol#L279 https://github.com/VANMofficial/Token-Sale/blob/1c0995c827b200c76e90efe3a809096a71251581/VANMToken.sol#L294-L295
Those lines may be deleted.
The function () payable { revert(); } was a pattern used to prevent implicit acceptance of ether in Solidity versions older than 0.4.0, but today this is unneeded.
No critical vulnerabilities were detected,but we highly recommend to complete this bugs before use.