Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: high
Valid

Leak of sensitive information

Summary

Anyone can read s_password state by analyze storage slot of this contract

Vulnerability Details

The s_password state is set private but it's actually not private because all states of smart contract are store on-chain and public to everyone.

Impact

If this password is used to protect some funds of other contracts the funds could be in danger.

Tools Used

Manual review

Recommendations

Should not store sensitive information on-chain

Updates

Lead Judging Commences

inallhonesty Lead Judge
about 2 years ago
inallhonesty Lead Judge about 2 years ago
Submission Judgement Published
Validated
Assigned finding tags:

finding-anyone-can-read-storage

Private functions and state variables are only visible for the contract they are defined in and not in derived contracts. In this case private doesn't mean secret/confidential

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.