Nouns DAO contest - asutorufos's results

A DAO-driven NFT project on Ethereum.

General Information

Platform: Code4rena

Start Date: 22/08/2022

Pot Size: $50,000 USDC

Total HM: 4

Participants: 160

Period: 5 days

Judge: gzeon

Total Solo HM: 2

Id: 155

League: ETH

Nouns DAO

Findings Distribution

Researcher Performance

Rank: 105/160

Findings: 1

Award: $35.44

🌟 Selected for report: 0

🚀 Solo Findings: 0

N-1 EVENT IS MISSING INDEXED FIELDS Each event should use three indexed fields if there are three or more fields

[NounsDAOInterfaces,sol L#50]https://github.com/code-423n4/2022-08-nounsdao/blob/c1c7c6201d0247f92472419ff657b570f9104565/contracts/governance/NounsDAOInterfaces.sol#:~:text=event%20ProposalCreatedWithRequirements(

NounsDAOInterfaces.sol L#70

NounsDAOInterfaces.sol L#120

N-2 Shows changes for an event before you change of vetoers address [NounsDAOLogicv2.sol l#839]https://github.com/code-423n4/2022-08-nounsdao/blob/c1c7c6201d0247f92472419ff657b570f9104565/contracts/governance/NounsDAOLogicV2.sol#:~:text=function%20_setVetoer(,%7D

N-3 USE A MORE RECENT VERSION OF SOLIDITY Use a solidity version of at least 0.8.13 to get the ability to use using for with a list of free functions

L-1 UNUSED/EMPTY RECEIVE()/FALLBACK() FUNCTION If the intention is for the Ether to be used, the function should call another function, otherwise it should revert NounsDAOLogicV@.sol L#1030

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