Submission Details
Severity:
loss of privileges
Summary
The "setPassword" function lacks proper access control, allowing anyone to call it and overwrite the existing password.
Vulnerability Details
The "setPassword" function lacks proper access control, allowing anyone to call it and overwrite the existing password.
function test_non_owner_setPassword() public {
vm.startPrank(address(1));
string memory attackPassword = "myattackPassword";
passwordStore.setPassword(attackPassword);
vm.startPrank(owner);
string memory password = passwordStore.getPassword();
assertEq(attackPassword , password);
}
Impact
This results in the loss of the old password.
Tools Used
vscode
Recommendations
Recommended to add permissions
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.