Submission Details
Severity:
setPassword doesn't check ownership allows anyone to set password
Summary
Anyone can set a new password via setPassword().
Vulnerability Details
No ownership check is made in setPassword().
Impact
High. The issue is easy to find and the attacker can obtain access to the contract.
Tools Used
Manual check.
Recommendations
Check if msg.sender == owner directly in setPassword() or create an onlyOwner() modifier and apply the modifier to setPassword(). Revert if msg.sender is not the owner.
Updates
Lead Judging Commences
inallhonesty
about 2 years ago
inallhonesty about 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.