Submission Details
Severity:
Data Integrity Attack due to lack of Access Control
Summary
The Attacker is able to change the password without them being the owner of the contract.
Vulnerability Details
There is no Check to see if the msg.sender is the owner in the function setPassword().
Impact
A Core-functionality of the contract is violated.
Tools Used
Manual Review.
Recommendations
Put a check to see if the msg.sender is the owner of the contract before line 27 in PasswordStore.sol before assigning the new password.
Updates
Lead Judging Commences
inallhonesty
almost 2 years ago
inallhonesty almost 2 years ago
Submission Judgement Published Assigned finding tags:
finding-lacking-access-control
Anyone can call `setPassword` and set a new password contrary to the intended purpose.
Rewards breakdown
nSLOC
20This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.