EigenLayer Contest - turvy_fuzz's results

Enabling restaking of staked Ether, to be used as cryptoeconomic security for decentralized protocols and applications.

General Information

Platform: Code4rena

Start Date: 27/04/2023

Pot Size: $90,500 USDC

Total HM: 4

Participants: 43

Period: 7 days

Judge: GalloDaSballo

Id: 233

League: ETH

EigenLayer

Findings Distribution

Researcher Performance

Rank: 36/43

Findings: 1

Award: $90.02

Gas:
grade-b

🌟 Selected for report: 0

🚀 Solo Findings: 0

Findings Information

Labels

bug
G (Gas Optimization)
grade-b
edited-by-warden
G-02

Awards

90.0161 USDC - $90.02

External Links

underlyingToken and podOwner variables can be made immutable

The variables underlyingToken and podOwner can be immutable

https://github.com/code-423n4/2023-04-eigenlayer/blob/main/src/contracts/strategies/StrategyBase.sol#L57 https://github.com/code-423n4/2023-04-eigenlayer/blob/main/src/contracts/pods/EigenPod.sol#L59

Pack bool hasRestaked with address podOwner(Saves 1 SLOT: 2k gas)

https://github.com/code-423n4/2023-04-eigenlayer/blob/main/src/contracts/pods/EigenPod.sol#L73

Redundant token == underlyingToken require check should be made modifier

https://github.com/code-423n4/2023-04-eigenlayer/blob/main/src/contracts/strategies/StrategyBase.sol#L86

copy totalShares value to memory for the deposit function as well

Just as done in the withdraw function: https://github.com/code-423n4/2023-04-eigenlayer/blob/main/src/contracts/strategies/StrategyBase.sol#L130

#0 - GalloDaSballo

2023-06-01T16:33:58Z

copy totalShares value to memory for the deposit function as well L

rest invalid -3

#1 - c4-judge

2023-06-01T16:59:05Z

GalloDaSballo marked the issue as grade-b

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