DatingDapp
First Flight #33
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Previous Next
Submission Details
Severity: low
Invalid

State variable could be declare immutable

nepker

Summary

State variable could be declare immutable

Vulnerability Details

State variables that are should be declared immutable to save gas. Add the immutable attribute to state variables that are only changed in the constructor

3 Found Instances

  • Found in src/LikeRegistry.sol Line: 15

    SoulboundProfileNFT public profileNFT;

  • Found in src/MultiSig.sol Line: 11

    address public owner1;

  • Found in src/MultiSig.sol Line: 12

    address public owner2;

Impact: It cost high gas fees

Tools Used :

Manual review+aderyn+vscodium

Recommendations:

Use immuatble instead of public

Updates

Appeal created

n0kto Lead Judge 7 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Informational or Gas

Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelyhood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.

Support

FAQs

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