Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wallet: encrypt the cache file #381

Merged
merged 2 commits into from
Aug 24, 2015

Conversation

moneromooo-monero
Copy link
Collaborator

It contains private data, such as a record of transactions.
The key is derived from the view secret key.

The encryption currently is one shot, so may require a lot of
memory for large wallet caches.

It contains private data, such as a record of transactions.
The key is derived from the view and spend secret keys.

The encryption currently is one shot, so may require a lot of
memory for large wallet caches.
@moneromooo-monero moneromooo-monero force-pushed the encrypt-cache branch 2 times, most recently from b09d8b7 to f197599 Compare August 22, 2015 21:10
@moneromooo-monero
Copy link
Collaborator Author

It won't update to the latest commit message, but: it now uses both the view and spend secret keys to generate the chacha key.

@fluffypony fluffypony merged commit f197599 into monero-project:master Aug 24, 2015
fluffypony added a commit that referenced this pull request Aug 24, 2015
f197599 wallet: encrypt the cache file (moneromooo-monero)
98c76a3 chacha8: add a key generation variant that take a pointer and size (moneromooo-monero)
scilicet64 pushed a commit to Beldex-Coin/beldex that referenced this pull request Mar 20, 2019
…oject#381)

* Fix switch fallthrough error after fixing export_transfers

* Change the assert to a warning
scilicet64 pushed a commit to Beldex-Coin/beldex that referenced this pull request Mar 20, 2019
…oject#381)

* Fix switch fallthrough error after fixing export_transfers

* Change the assert to a warning
scilicet64 pushed a commit to Beldex-Coin/beldex that referenced this pull request Mar 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants