Skip to content

Instantly share code, notes, and snippets.

@danbogd

danbogd/BinanceSmartChain_SHIELD.md Secret

Created February 7, 2021 18:37
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 danbogd/c3d990fc32ccd1623e2a9898fd636904 to your computer and use it in GitHub Desktop.
Save danbogd/c3d990fc32ccd1623e2a9898fd636904 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
BinanceSmartChain_SHIELD.md

BinanceSmartChain_SHIELD Audit Report.

1. Summary

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

2. In scope

3. Findings

In total,2 issues were reported including:

  • 1 low severity issues.
  • 1 notes

No critical security issues were found.

3.1. Known vulnerabilities of ERC-20 token

Severity: low

Description

  1. Lack of transaction handling mechanism issue. WARNING! This is a very common issue and it already caused millions of dollars losses for lots of token users! More details here.

3.2. Use the lates version contracts from openZeppelin

Severity: note

Description

If you want to use the pragma solidity 0.7.6, I recommend you to upgrade your Ownable and Context contracts to the lates version contracts from openZeppelin library using abstract and the virtual keyword.

4. Conclusion

The review did not show any critical issues, some low severity issues and notes were found.

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