Nested Finance contest - hansfriese's results

The one-stop Defi app to build, manage and monetize your portfolio.

General Information

Platform: Code4rena

Start Date: 15/06/2022

Pot Size: $35,000 USDC

Total HM: 1

Participants: 36

Period: 3 days

Judge: Jack the Pug

Total Solo HM: 1

Id: 137

League: ETH

Nested Finance

Findings Distribution

Researcher Performance

Rank: 25/36

Findings: 1

Award: $81.82

🌟 Selected for report: 0

🚀 Solo Findings: 0

Awards

81.8216 USDC - $81.82

Labels

bug
QA (Quality Assurance)
valid

External Links

Non-critical Issues

  1. Constants should be defined rather than using magic numbers 2022-06-nested\contracts\NestedFactory.sol#L161 2022-06-nested\contracts\NestedFactory.sol#L169 2022-06-nested\contracts\NestedFactory.sol#L264 2022-06-nested\contracts\NestedFactory.sol#L378 2022-06-nested\contracts\NestedFactory.sol#L443 2022-06-nested\contracts\NestedFactory.sol#L629

  2. Natspec incomplete 2022-06-nested\contracts\NestedFactory.sol#L114 @return

2022-06-nested\contracts\operators\Beefy\lp\BeefyZapBiswapLPVaultOperator.sol#L232 @return

2022-06-nested\contracts\operators\Beefy\lp\BeefyZapBiswapLPVaultOperator.sol#L264 @param reserveA @param reserveB @param router @return

2022-06-nested\contracts\operators\Beefy\lp\BeefyZapUniswapLPVaultOperator.sol#L263 @param reserveA @param reserveB @param router @return

  1. Typo 2022-06-nested\contracts\NestedFactory.sol#L51 2022-06-nested\contracts\NestedFactory.sol#L639

Change "withdrawed" to "withdrew"

  1. Immutable addresses should be 0-checked 2022-06-nested\contracts\Withdrawer.sol#L17 2022-06-nested\contracts\operators\Yearn\YearnCurveVaultOperator.sol#L48-L50

#0 - Yashiru

2022-06-24T13:16:04Z

1. Constants should be defined rather than using magic numbers (Duplicated)

Duplicated of #76 at 5. constants should be defined rather than using magic numbers

#1 - Yashiru

2022-06-24T14:18:42Z

3. Typo (Duplicated)

Duplicated of #45 at Typos

#2 - obatirou

2022-06-24T16:00:09Z

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