Cally contest - bobirichman's results

Earn yield on your NFTs or tokens via covered call vaults.

General Information

Platform: Code4rena

Start Date: 10/05/2022

Pot Size: $50,000 USDC

Total HM: 13

Participants: 100

Period: 5 days

Judge: HardlyDifficult

Total Solo HM: 1

Id: 122

League: ETH

Cally

Findings Distribution

Researcher Performance

Rank: 53/100

Findings: 2

Award: $85.03

🌟 Selected for report: 0

🚀 Solo Findings: 0

QA Report

Table of content

QA Findings

Multiplications should come before divisions to improve precision

You loss precision because of doing divisions before the multiplication. If you change the order it will improve the precision at no cost.

Gas Report

Table of content

Gas Optimizations

Change transferFrom to transfer

transferFrom(address(this), , **)' could be replaced by the following more gas efficient 'transfer(, **)'. This replacement is more gas efficient and improves the code quality.

State variables that could be set immutable

You can set the following state variables to immutable and save gas:

Unnecessary equals boolean

Boolean variables can be checked within conditionals directly without the use of equality operators to true/false.

AuditHub

A portfolio for auditors, a security profile for protocols, a hub for web3 security.

Built bymalatrax © 2024

Auditors

Browse

Contests

Browse

Get in touch

ContactTwitter