Submission Details
Severity:
getPassword function Optimized
Summary
As i used onlyOwner modifier and ABI's encoding utility that's why i have to modify & optimize the getPassword function.
Vulnerability Details
function getPassword() external view onlyOwner returns (string memory) {
// I removed this check. Since, I'm using onlyOwner modifier. Empowering DRY. 😁
// if (msg.sender != s_owner) {
// revert PasswordStore__NotOwner();
// }
// return s_password; // I used abi's encoding that's why i first decoded password then returned it.
return abi.decode(s_password, (string));
}
Impact
Optimizes the source code and makes retrieval more secure.
Tools Used
Manually optimized.
Recommendations
We Should follow DRY and Best Practices to retrieve password securely.
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.