Skip to content

Instantly share code, notes, and snippets.

@MrCrambo
Created January 28, 2019 10:47
Show Gist options
  • Save MrCrambo/65b70c8db86bb2f5fd9653de055b1a73 to your computer and use it in GitHub Desktop.
Save MrCrambo/65b70c8db86bb2f5fd9653de055b1a73 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