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

cancelRequest():: Anyone without authorization can cancel the request.

Summary

No check to prevent unauthorized person from cancelling request.

Vulnerability Details

Inside Bridge.sol, In function cancelRequest() there is no access control modifier to ensure only authorized address can cancel the request.

Impact

Request can be canceled by anyone not having authorization.

Tools Used

Manual

Recommendations

Add onlyOwner modifier in function cancelRequest().

Updates

Lead Judging Commences

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

invalid-cancel-callable-by-anyone

The process to cancel a message is detailed here: https://docs.starknet.io/architecture-and-concepts/network-architecture/messaging-mechanism/#l2-l1_message_cancellation Since `startRequestCancellation` has the `onlyOwner`, only the owner can begin that process.

Support

FAQs

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