Submission Details
Severity:
Missing access control
Summary
Missing access control is setPassword()
Vulnerability Details
According to dev comment * @notice This function allows only the owner to set a new password. but there is no access control modifier or require statement that current user is owner of the contract.
Impact
Anyone can hit setPassword() and set new password.
Tools Used
Manual code review
Recommendations
Use modifier or make require statement for checking that user must be owner
function setPassword(string memory newPassword) external {
+ require(msg.sender == s_owner, "not owner");
s_password = newPassword;
emit SetNetPassword();
}
Updates
Lead Judging Commences
inallhonesty
over 1 year ago
inallhonesty over 1 year 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.