Sparkn

CodeFox Inc.
DeFiFoundryProxy
15,000 USDC
View results
Submission Details
Severity: medium
Valid

Missing check for `address(0)` when assigning values to `address` state variables

Summary

Lack of zero-address validation on address parameters may lead to transaction reverts, wastes gas, may require resubmission of transactions, and may force contract redeployments in certain cases within the protocol.

Vulnerability Details

There is 1 instance of this issue.

File: src/Proxy.sol
45: _implementation = implementation;
File Link Instance Count Instance Link
Proxy.sol 1 45

Impact

The proxy will be unusable and will require a new proxy to be created.

Tools Used

baudit: a custom static code analysis tool; manual review

Recommendations

Consider adding explicit zero-address validation prior to assignment of a value to an address state variable.

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.