Skip to content

Instantly share code, notes, and snippets.

@danbogd

danbogd/BCT_audit_report.md

Created October 23, 2018 11:19
Show Gist options
  • Save danbogd/e484a0ef5f4b5afe655ce1eae5c4c178 to your computer and use it in GitHub Desktop.
Save danbogd/e484a0ef5f4b5afe655ce1eae5c4c178 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
BCT_audit_report.md

BCT Audit Report.

1. Summary

This document is a security audit report performed by danbogd, where BCT has been reviewed.

2. In scope

3. Findings

In total, 1 issues were reported including:

  • 1 low severity issues.

No critical security issues were found.

3.1. No checking for zero address.

Severity: low

Description

Function transfer member of BCT contract do not require the to address to be non null before transfer. Accidental token loss to address 0x0 can be applicable.

Code snippet

https://gist.github.com/yuriy77k/341c72bb372a30a98df9ebd7797ecac1#file-bct-sol-L239

4. Conclusion

No critical vulnerabilities were detected, but we highly recommend to complete this bugs before use.

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