Skip to content

Instantly share code, notes, and snippets.

@yuriy77k
Forked from MrCrambo/ETH_OmiseGo_report.md
Created March 27, 2019 15: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/960e5a709d35522a5deb3f35c4ba15b3 to your computer and use it in GitHub Desktop.
Save yuriy77k/960e5a709d35522a5deb3f35c4ba15b3 to your computer and use it in GitHub Desktop.

Summary

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

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

In scope

  1. https://gist.github.com/yuriy77k/d52b7674ca7cf9b221a819db168fb645

Findings

In total, 3 issues was reported including:

  • 0 critical severity issues.

  • 0 high severity issues.

  • 1 medium severity issues.

  • 2 low severity issues.

1. Zero address

Severity: low

Description

There is no zero address checking at functions transfer and transferFrom.

2. Owner privilegies

Severity: low

Description

Owner can pause at any time he wants.

3. There is no minting cap

Severity: medium

Description

Owner can mint any amount he wants and there is no hard cap.

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