Nouns Builder contest - Captainkay's results

A permissionless, governed protocol to deploy nouns-style DAOs complete with treasury, generative collections, and governance mechanisms.

General Information

Platform: Code4rena

Start Date: 06/09/2022

Pot Size: $90,000 USDC

Total HM: 33

Participants: 168

Period: 9 days

Judge: GalloDaSballo

Total Solo HM: 10

Id: 157

League: ETH

Nouns Builder

Findings Distribution

Researcher Performance

Rank: 138/168

Findings: 1

Award: $60.77

🌟 Selected for report: 0

🚀 Solo Findings: 0

Lines of code

https://github.com/code-423n4/2022-09-nouns-builder/blob/main/src/token/Token.sol#L43-L66

Vulnerability details

Impact

In the Code For Token.sol the token's auction house Initiator(...,address _auction) doesnt checked for non-zero addressess, this can lead to loss of tokes if user/sent to that address

Reference https://github.com/elasticdao/contracts/pull/78

Proof of Concept

https://github.com/code-423n4/2022-09-nouns-builder/blob/main/src/token/Token.sol#L43-L66 No check for

Tools Used

N/A

have a check if (_auction == address(0)) revert ADDRESS_ZERO();

#0 - GalloDaSballo

2022-09-16T19:11:31Z

Agree with the finding, disagree with severity, zero-checks are low historically

Also misconfiguration will force a re-deploy per mint and burn

#1 - GalloDaSballo

2022-09-28T22:51:15Z

L

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