Skip to content

Instantly share code, notes, and snippets.

@danbogd

danbogd/Farm.md

Created November 23, 2018 11:42
Show Gist options
  • Save danbogd/796e9624b5d45d621a88d739dd2dd8d4 to your computer and use it in GitHub Desktop.
Save danbogd/796e9624b5d45d621a88d739dd2dd8d4 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
Farm.md

V2 - P3C.io Auto-Reinvestment Farm audit report.

1. Summary

This document is a security audit report performed by danbogd, where V2 - P3C.io Auto-Reinvestment Farm has been reviewed.

2. In scope

3. Findings

In total, 2 issues were reported including:

  • 2 low severity issues.

No critical security issues were found.

3.1. No checking for zero address.

Severity: low

Description

Transfer function and constructor Crop members of Crop contract do not require the to address to be non null before transfer.

3.2. Known vulnerabilities of ERC-20 token

Severity: low

Description

ERC20 Tokens have some well-known issues (listed below), This is just a reminder for the contract developers.

  1. Lack of transaction handling mechanism issue.

The above mentioned issues are well documented, a basic search can help to get more information.

4. Conclusion

No critical vulnerabilities were detected,but we highly recommend to complete this bugs before use.

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