Submission Details
Severity:
getPassword function should emit an event
Summary
It might be a good idea to emit an event while getting password as well to keep a record of who accessed the password data.
Vulnerability Details
New event might be added to emit it while getting password.
Impact
Emitting an event in the getter function is considered a good practice and may allow to keep a record of accessing the password data.
Tools Used
Manual codebase analysis.
Recommendations
+ event GetPassword();
function getPassword() external view returns (string memory) {
if (msg.sender != s_owner) {
revert PasswordStore__NotOwner();
}
+ emit GetPassword();
return s_password;
}
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.