code-423n4 / 2022-10-thegraph-findings

0 stars 0 forks source link

No check for zero address #250

Closed code423n4 closed 1 year ago

code423n4 commented 1 year ago

Lines of code

https://github.com/code-423n4/2022-10-thegraph/blob/main/contracts/governance/Managed.sol#L173-L180

Vulnerability details

Impact

No check for zero address

Proof of Concept

https://github.com/code-423n4/2022-10-thegraph/blob/main/contracts/governance/Managed.sol#L173-L180

Tools Used

Recommended Mitigation Steps

Add check for zero address for contractAddress inside there is not need to emit event if contract does not exist.