Skip to content

Instantly share code, notes, and snippets.

@yuriy77k

yuriy77k/Atomz_conclusion.md

Created September 25, 2018 17:43
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save yuriy77k/a708e10f16772b0014b4b8b5cffed851 to your computer and use it in GitHub Desktop.
Save yuriy77k/a708e10f16772b0014b4b8b5cffed851 to your computer and use it in GitHub Desktop.
Download ZIP
Atomz audit report
Raw
Atomz_conclusion.md

Atomz security audit report performed by Callisto Security Audit Department

1. Conclusion:

The token contract can be considered safe to be deployed.

2. Low severity issues:

2.1. Known Issues of ERC20 Standard

Description

ERC20 Tokens have some well-known issues (listed bellow), This is just a reminder for the contract developers.

  • Approve + transferFrom mechanism allows double Withdrawal attack. More details here
  • Lack of transaction handling. More details here

3. Minor observation:

3.1. Extra checking

Description

Extra checking in 92 line. SafeMath library checks it anyway.

Recommendation

This line may be deleted.

Revealing audit reports:

https://gist.github.com/yuriy77k/e4ea968e4f2a4475a5aec0433b9d0e23

https://gist.github.com/yuriy77k/5a14fc9dbc29c1129c4517bbf9bf5162

https://gist.github.com/yuriy77k/dba77a5bcd7f9d28b6246b1ae413efd1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment