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
Rank: 36/43
Findings: 1
Award: $90.02
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: neutiyoo
Also found by: 0xSmartContract, 0xnev, Aymen0909, QiuhaoLi, ReyAdmirado, clayj, ihtishamsudo, naman1778, niser93, pontifex, tonisives, turvy_fuzz
90.0161 USDC - $90.02
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
https://github.com/code-423n4/2023-04-eigenlayer/blob/main/src/contracts/pods/EigenPod.sol#L73
token == underlyingToken
require check should be made modifiertotalShares
value to memory for the deposit function as wellJust 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