Closed yuriy77k closed 5 years ago
Auditing time: 1 day.
@RideSolo assigned
Auditing time 1 day
Estimated auditing time is 1 day.
My report is finished.
@MrCrambo @gorbunovperm assigned
ExploreCoin smart contract security audit report performed by Callisto Security Audit Department
In total, 4 issues were reported including:
3 low severity issues.
1 notes.
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.
Add the following code to the transfer(_to address, ...)
function:
require( _to != address(this) );
decreaseAllowance
throw in case if the value to be substracted is higher than the amount that is allowed, if the address owner wants to change the value allowed by reducing it and the spender withdraw a part of it before, the function might throw and give more chances for the spender to take the rest of the allowed value.
The value should be set to zero if the value to be subtracted is higher than the allowance.
In constructor
at line 306 there have to be zero address checking for feeReceiver
and tokenOwnerAddress
.
Functions _burn
and _burnFrom
are internal and unused.
The audited smart contract can be deployed. Only low severity issues were found during the audit.
https://gist.github.com/yuriy77k/18e58f1d87b9cf05db92b89b7512b87f
https://gist.github.com/yuriy77k/0e650f3827e33abcd9f45a68e71f03e4
https://gist.github.com/yuriy77k/13fb50504be43071bd55615ee38be669
Thanks a lot for the audit.
Audit request
ExploreCoin (EPL) is an ERC 20 token, smart contract recently deployed.
https://www.explorecoin.io/
Source code
https://etherscan.io/address/0x2802faad8a0f9595c1660c436c7e51d9ce97b1e9#code
Disclosure policy
admin@explorecoin.io
Platform
ETH
Number of lines:
128