NFTBridge
60,000 USDC
View results
Submission Details
Severity: low
Invalid

Unused address in escrow mapping

Summary

Unused address which is set when adding to escrow - _escrow[collection][id] = msg.sender

Vulnerability Details

Can be just marked as true/false because in case of l2->l1 bridging, l2 owner can set any address on l1 as the new owner.

Impact

Tools Used

Recommendations

Set _escrow[collection][id] mapping value to a simple true/false

Updates

Lead Judging Commences

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

Informational / Gas

Please, do not suppose impacts, think about the real impact of the bug and check the CodeHawks documentation to confirm: https://docs.codehawks.com/hawks-auditors/how-to-determine-a-finding-validity A PoC always helps to understand the real impact possible.

Support

FAQs

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