Submission Details
Severity:
the comment describing the getPassword function are incorrect
Summary
The comment describing the PasswordStore::getPassword function is incorrect, it indicates sending a parameter that is not used by the function
Vulnerability Details
The comment indicates a parameter newPassword to pass to the function, but the PasswordStore::getPassword function has no parameters.
The comment was probably copied from the PasswordStore::setPassword function and the line indicating the parameter to pass to the function was not removed.
/*
* @notice This allows only the owner to retrieve the password.
@> * @param newPassword The new password to set.
*/
function getPassword() external view returns (string memory) {
if (msg.sender != s_owner) {
revert PasswordStore__NotOwner();
}
return s_password;
}
Impact
The impact is minimal because it does not block the functioning of the smart contract.
Tools Used
Manual check
Recommendations
Update the comment removing the @param row.
Updates
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.