Project

One World
NFTDeFi
15,000 USDC
View results
Submission Details
Severity: low
Invalid

`proxyAdmin` and `owpWallet` can be declared as `immutable`

Summary

In MembershipFactory.sol variables proxyAdmin and owpWallet can be declared as immutable.

Vulnerability Details

In MembershipFactory contract variables proxyAdmin and owpWallet are set only in the constructor so they can be declared as immutable.

Impact

immutable variables consume less gas compared to regular state variables because they are stored directly in the contract’s bytecode rather than the storage, making reads from these variables cheaper.

Tools Used

Slither, manual review

Recommendations

Declare variables as immutable:

- ProxyAdmin public proxyAdmin;
- address public owpWallet;
+ ProxyAdmin public immutable proxyAdmin;
+ address public immutable owpWallet;
Updates

Lead Judging Commences

0xbrivan2 Lead Judge 7 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
0xbrivan2 Lead Judge 7 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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