code4rena-2022-08-nounsdao-l03
[L‑03] Missing checks for address(0x0) when assigning values to address state variables
Summary
address 변수에 값을 할당할 때 address(0) 인지 확인하지 않았음을 지적
Keyword
input validation, address
Vulnerability
- contracts/governance/NounsDAOLogicV1.sol#L605
- contracts/governance/NounsDAOLogicV2.sol#L807
- contracts/governance/NounsDAOProxy.sol#L71
File: contracts/governance/NounsDAOLogicV1.sol
605: pendingAdmin = newPendingAdmin;
642: vetoer = newVetoer;File: contracts/governance/NounsDAOLogicV2.sol
807: pendingAdmin = newPendingAdmin;
844: vetoer = newVetoer;File: contracts/governance/NounsDAOProxy.sol
71: admin = admin_;address 변수에 값을 할당할 때, address(0) 인지 확인하지 않았다는 점을 지적했다.
컨트랙트 주소의 경우 특히 주의할 필요가 있으므로 인정되는것 같다.
Impact
실수로 잘못된 주소를 변수에 저장했을 시 컨트랙트가 오작동할 수 있다.
Mitigation
address(0) 인지 확인한다.
Memo
별 것 아닌 것 같은데, 이런것도 상금을 주긴 하는 것 같다. address(0) 만 확인하는게 능사가 아님에도..
tags: bughunting, nouns dao, lack-of-input-validation-vul, smart contract, solidity, severity low