Enable leveldb obfuscation for new databases #7923

pull kazcw wants to merge 1 commits into bitcoin:master from kazcw:obfuscate changing 1 files +1 −1

kazcw commented at 2:00 AM on April 22, 2016: contributor

This was implemented some time ago in #6650 to fix #6613 but does not appear ever to have been enabled under any circumstances outside of test/.

On a recent fresh install I'm seeing a null encryption key, and I don't see anywhere a CDBWrapper is ever created with a non-default obfuscate parameter (the default is false).

Enable leveldb obfuscation for new databases

This was implemented some time ago in #6650 to fix #6613 but does not appear
ever to have been enabled under any circumstances outside of test/

7a539dec7c

gmaxwell commented at 2:04 AM on April 22, 2016: contributor

Only the chainstate database is obfscuated because only it contains non-trivial amounts of potentially malicious input.

CCoinsViewDB::CCoinsViewDB(size_t nCacheSize, bool fMemory, bool fWipe) : db(GetDataDir() / "chainstate", nCacheSize, fMemory, fWipe, true)

Also, I see it fine in logs:

2016-04-18 19:08:12 Using obfuscation key for /root/.bitcoin/chainstate: de0e9b59430e40f1
kazcw commented at 2:08 AM on April 22, 2016: contributor

Oh, false alarm. I must have had old .bitcoin files on this machine and missed that instantiation in the grep.
kazcw closed this on Apr 22, 2016
kazcw deleted the branch on Apr 22, 2016
MarcoFalke locked this on Sep 8, 2021

Contributors