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

s_password will be public for anyone to see

Summary

Data stored on the blockchain is visible to anyone. Storing sensitive data such as a password is not advised.

Vulnerability Details

The storage variable s_password stores sensitive data, this leads to sensitive data leak since all storage data is public.

Impact

Sensitive data leak.

cast storage $PASSWORD_STORE_CONTRACT_ADDRESS 1 --rpc-url $RPC_URL

Tools Used

  • Foundry

  • Slither

Recommendations

  • Do not store sensitive data on the blockchain.

  • If sensitive data needs to be stored on the blockchain, consider encrypting it first.

Updates

Lead Judging Commences

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.