Yieldy contest - 0xc0ffEE's results

A protocol for gaining single side yields on various tokens.

General Information

Platform: Code4rena

Start Date: 21/06/2022

Pot Size: $50,000 USDC

Total HM: 31

Participants: 99

Period: 5 days

Judges: moose-code, JasoonS, denhampreen

Total Solo HM: 17

Id: 139

League: ETH

Yieldy

Findings Distribution

Researcher Performance

Rank: 75/99

Findings: 1

Award: $54.61

🌟 Selected for report: 0

🚀 Solo Findings: 0

  1. Duplicate external call IERC20Upgradeable(YIELDY_TOKEN).approve(LIQUIDITY_RESERVE,type(uint256).max) in function initialize of Staking.sol
  2. Consider reorder order for isStakingPaused, isUnstakingPaused, isInstantUnstakingPaused in StakingStorage.sol for better storage slot packing to save gas
  3. The use of arguments in _storeRebase(updatedTotalSupply, _profit, _epoch) in rebase(uint256,uint256) (from Yieldy.sol) lead to totalStakedBefore and totalStakedAfter are identical
  4. variable MAX_UINT256 in YieldyStorage.sol is unused
  5. transferFrom(address,address,uint256) in Yieldy.sol should check for _to != address(0) to prevent locking funds at zero address
  6. Use unchecked for gas saving at https://github.com/code-423n4/2022-06-yieldy/blob/524f3b83522125fb7d4677fa7a7e5ba5a2c0fe67/src/contracts/Yieldy.sol#L192 , https://github.com/code-423n4/2022-06-yieldy/blob/524f3b83522125fb7d4677fa7a7e5ba5a2c0fe67/src/contracts/Yieldy.sol#L212 and https://github.com/code-423n4/2022-06-yieldy/blob/524f3b83522125fb7d4677fa7a7e5ba5a2c0fe67/src/contracts/Yieldy.sol#L288-L290
  7. https://github.com/code-423n4/2022-06-yieldy/blob/524f3b83522125fb7d4677fa7a7e5ba5a2c0fe67/src/contracts/Yieldy.sol#L217 should have a require() for revert reason if underflow
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