Skip to content

Instantly share code, notes, and snippets.

@yuriy77k
Forked from MrCrambo/ETH_Aurora_report.md
Created July 6, 2019 10:59
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/1139b269b38e984a8189a9713e8ea051 to your computer and use it in GitHub Desktop.
Save yuriy77k/1139b269b38e984a8189a9713e8ea051 to your computer and use it in GitHub Desktop.

Summary

This is the report from a security audit performed on Aurora by MrCrambo.

The audit focused primarily on the security of Aurora smart contract.

In scope

  1. https://etherscan.io/address/0x9ab165d795019b6d8b3e971dda91071421305e5a#contracts

Findings

In total, 4 issues were reported including:

  • 0 high severity issues.

  • 1 medium severity issues.

  • 0 owner privilegies issues.

  • 2 low severity issues.

  • 1 notes.

Security issues

1. Zero address chekcing

Severity: low

Description

There are no zero address checking in functions transfer and transferFrom.

2. Known vulnerabilities of ERC-20 token

Severity: low

Description

  1. It is possible to double withdrawal attack. More details here
  2. Lack of transaction handling mechanism issue. More details here

Recommendation

Add into a function transfer(address _to, ... ) following code:

require( _to != address(this) );

3. Wrong transfering

Severity: medium

Description

Functions transfer and transferFrom return false instead of throwing, in case of transfer failing.

4. No event call

Severity: note

Description

There is no event call in constructor where all funds sent to the owner.

Conclusion

Smart contract contains medium severity issue.

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