Skip to content

Instantly share code, notes, and snippets.

@yuriy77k yuriy77k/Snowflake.md Secret forked from MrCrambo/Snowflake.md
Created Jan 28, 2019

Embed
What would you like to do?

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
You can’t perform that action at this time.