PoolTogether Aave v3 contest - throttle's results

A protocol for no loss prize savings on Ethereum.

General Information

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

PoolTogether

Findings Distribution

Researcher Performance

Rank: 28/40

Findings: 1

Award: $39.05

🌟 Selected for report: 0

🚀 Solo Findings: 0

Awards

39.0496 USDC - $39.05

Labels

bug
G (Gas Optimization)

External Links

[N-01] Unused SafeMath

Description

using SafeMath for uint256 is not needed since 0.8.0 solidity version.

https://github.com/pooltogether/aave-v3-yield-source/blob/e63d1b0e396a5bce89f093630c282ca1c6627e44/contracts/AaveV3YieldSource.sol#L26

SafeMath usage is a no-op and yet it makes code less clear.

Remove safe math usage

-----------------------------------------------------------------

[N-02] Bump packages

Description

4.6.0 OpenZeppelin version is available 1.16.0 @aave/core-v3 version is available 1.17.0 @aave/periphery-v3 version is available

Bump packages accordingly

-----------------------------------------------------------------

#0 - PierrickGT

2022-05-03T22:12:14Z

[N-01] Unused SafeMath

Duplicate of https://github.com/code-423n4/2022-04-pooltogether-findings/issues/11

[N-02] Bump packages

This is not really part of the audit scope.

#1 - gititGoro

2022-05-14T03:57:32Z

Safemath removal is a gas optimization issue

#2 - JeeberC4

2022-05-23T16:33:21Z

Changing name to better reflect the judging outcome.

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