Skip to content

Instantly share code, notes, and snippets.

@yuriy77k
Forked from MrCrambo/Snowflake.md
Created January 28, 2019 12:33
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/c8e0997f2cee239d6ab35c28710e4952 to your computer and use it in GitHub Desktop.
Save yuriy77k/c8e0997f2cee239d6ab35c28710e4952 to your computer and use it in GitHub Desktop.

Summary

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

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

In scope

  1. https://github.com/HydroBlockchain/smart-contracts/blob/master/snowflake/contracts/Snowflake.sol

Findings

In total, 1 issue were reported including:

  • 0 high severity issues.

  • 0 medium severity issues.

  • 1 low severity issues.

Security issues

1. No zero address checking

Severity: low

Description

There is no zero address checking at function setAddresses.

Recommendation

Add zero address checking

require(_identityRegistryAddress != address(0) && _hydroTokenAddress != address(0));

Conclusion

Smart contract has only low severity issue.

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