First Flight #12: Kitty Connect

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

State changes followed by no event emissions in `KittyBridge` contract

Summary

State changes followed by no event emissions in KittyBridge contract

Vulnerability Details

In the KittyBridge contract, in allowlistDestinationChain, allowlistSourceChain and allowlistSender, we add in the allowlist mappings but we don't emit any event for this state change.

Impact

This can lead to a lack of transparency and can make it difficult to track the changes in the state of the contract.

Tools Used

Manual review

Recommendations

Consider emitting this events after the state changes:

+ event DestChainAdded(chainId, true);
+ event SourceChainAdded(chainId, true);
+ event SenderAdded(sender, true);
Updates

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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