Platform: Code4rena
Start Date: 29/04/2022
Pot Size: $22,000 USDC
Total HM: 6
Participants: 40
Period: 3 days
Judge: Justin Goro
Total Solo HM: 2
Id: 114
League: ETH
Rank: 39/40
Findings: 1
Award: $27.86
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: IllIllI
Also found by: 0v3rf10w, 0x1f8b, 0x4non, 0xDjango, 0xf15ers, 0xkatana, 242, Dravee, GimelSec, MaratCerby, Tadashi, TrungOre, WatchPug, defsec, fatherOfBlocks, gzeon, hake, horsefacts, joestakey, miguelmtzinf, pauliax, pedroais, peritoflores, rotcivegaf, simon135, slywaters, tabish, throttle, z3s
27.8625 USDC - $27.86
Make 3 indexed variables to save gas https://github.com/pooltogether/aave-v3-yield-source/blob/0f52c5ff27e7c2ef1b32fe2fb12ac16b1bba5e75/contracts/AaveV3YieldSource.sol#L41 https://github.com/pooltogether/aave-v3-yield-source/blob/0f52c5ff27e7c2ef1b32fe2fb12ac16b1bba5e75/contracts/AaveV3YieldSource.sol#L58 https://github.com/pooltogether/aave-v3-yield-source/blob/0f52c5ff27e7c2ef1b32fe2fb12ac16b1bba5e75/contracts/AaveV3YieldSource.sol#L66 https://github.com/pooltogether/aave-v3-yield-source/blob/0f52c5ff27e7c2ef1b32fe2fb12ac16b1bba5e75/contracts/AaveV3YieldSource.sol#L75
#0 - PierrickGT
2022-05-03T23:13:11Z
I think it's worthwhile to index the owner address in AaveV3YieldSourceInitialized event but as a rule of thumb, it's best to only index addresses that may be relevant if we need to build a subgraph.
#1 - PierrickGT
2022-05-03T23:14:41Z