Submission Details
Severity:
`setPassword` lacks an access control check, allowing anyone to set a new password
Summary
setPassword lacks an access control check, allowing anyone to set a new password
Vulnerability Details
/*
* @notice This function allows only the owner to set a new password.
* @param newPassword The new password to set.
*/
function setPassword(string memory newPassword) external {
s_password = newPassword;
emit SetNetPassword();
}
The setPassword is intended to allow only the owner to set a new password.
However, there is no access control check implemented in the function. Anyone can call this function and set a new password for the contract.
Impact
Everyone can set a new password.
Tools Used
Manual Review.
Recommendations
It should include an access control check within the setPassword function.
require(msg.sender == s_owner, "not owner!");
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.