Skip to content

Instantly share code, notes, and snippets.

@yuriy77k

yuriy77k/ETH_ERC223standardimplementation_audit_report.md

Forked from danbogd/ETH_ERC223standardimplementation_audit_report.md
Created July 7, 2019 15:34
Show Gist options
  • Star 1 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save yuriy77k/f45eca85fe21d3060b6861db7e2920e2 to your computer and use it in GitHub Desktop.
Save yuriy77k/f45eca85fe21d3060b6861db7e2920e2 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
ETH_ERC223standardimplementation_audit_report.md

ERC223 standard implementation for openzeppelinaudit report.

1. Summary

This document is a security audit report performed by danbogd, where ERC223 standard implementation for openzeppelin has been reviewed.

2. In scope

Сommit hash .

3. Findings

In total, 2 issues were reported including:

  • 1 medium severity issues
  • 0 low severity issues
  • 0 owner privileges (ability of owner to manipulate contract, may be risky for investors)..
  • 1 notes.

No critical security issues were found.

3.1. Typo.

Severity: low/medium

Description

Put a semicolon at the end of transfer function.

Code snippet

https://github.com/Sparke2/openzeppelin-solidity/blob/07e92b9a90c31381e512d0bf5e72934ef2daf02d/contracts/token/ERC223/IERC223.sol#L35

3.2. Extra code.

Severity: note

Description

Functions of this file (address.sol) is not used in this contract.

Code snippet

https://github.com/Sparke2/openzeppelin-solidity/blob/07e92b9a90c31381e512d0bf5e72934ef2daf02d/contracts/token/ERC223/ERC223.sol#L6

Recommendation

This line may be deleted.

4. Conclusion

The review did not show any critical issues, one medium or low severity issue was found.

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