First Flight #12: Kitty Connect

First Flight #12: Kitty Connect
Beginner FriendlyFoundryNFTGameFi
100 EXP
View results
Submission Details
Severity: low
Valid

Ownership immutability

Summary

The kittyConnect.sol::i_kittyConnectOwner variable is currently declared as immutable, preventing future ownership transfers.

Vulnerability Details

Immutable variables, once set during deployment, remain unchanged thereafter. If the owner's wallet is compromised or lost, critical functionalities may be impacted such as adding new shops, source and destinations chains or senders to the allow lists.

Tools Used

Manual review.

Recommendations

Consider declaring the variable as mutable. Additionally, implement a function allowing ownership transfers, restricted to only the current owner's invocation.

Updates

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
robertodf99 Submitter
over 1 year ago
inallhonesty Lead Judge
over 1 year ago
robertodf99 Submitter
over 1 year ago
inallhonesty Lead Judge
over 1 year ago
inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

Owner can't transfer ownership as Contest Details say.

Support

FAQs

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