Skip to content

Instantly share code, notes, and snippets.

@yuriy77k yuriy77k/ Secret

forked from MrCrambo/
Created Mar 17, 2019
What would you like to do?


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

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

In scope



In total, 2 issues was reported including:

  • 0 critical severity issues.

  • 0 high severity issues.

  • 0 medium severity issues.

  • 2 low severity issues.

1. Known vulnerabilities of ERC-20 token

Severity: low


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


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

require( _to != address(this) );

2. Old solidity version

Severity: low


Used solidity version is old.


Smart contract contains only low severity issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.