Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: medium
Invalid

The function validateTransaction does not check when validating the owner

Summary
The validation function does not check whether the user calling the contract is the owner or not.

Vulnerability Details
The vulnerability is that an attacker can indicate his Paymaster and assign the transaction to himself, line 4 is missing owner check. There is a need for BootLoader, an attacker can bypass the bootloader “check”
'''

function validateTransaction(bytes32, /*_txHash*/ bytes32, /*_suggestedSignedHash*/ Transaction memory _transaction)
external
payable
requireFromBootLoader
returns (bytes4 magic)
{
return _validateTransaction(_transaction);
}

'''

Impact

There may be gas manipulation due to the fact that the attacker has become the “master” and uses validation

Tools Used VSCode, GIT, Foundry, Searching in WEB about BootLoader

Recommendations "Add OrOwner"

Updates

Lead Judging Commences

bube Lead Judge 12 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Appeal created

dmitry17 Submitter
12 months ago
bube Lead Judge
12 months ago
bube Lead Judge 12 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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