Submission Details
Severity:
Password can be overwritten by anyone
Summary
The stored password can be overwritten by unauthorized users and will be lost forever.
Impact
There is no check if its the owner calling the setPassword() function. Anyone can therefor change the password.
Tools Used
Foundry
Recommendations
Check for owner in the setPassword() function
function setPassword(string memory newPassword) external {
if (msg.sender != s_owner)
{
revert PasswordStore__NotOwner();
}
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.