Allow the default key to be unavailable #2566

pull sipa wants to merge 1 commits into bitcoin:master from sipa:nodef changing 4 files +32 −24

sipa commented at 5:36 PM on April 25, 2013: member

This solves the issue where no default key can be added after -salvagewallet.

Allow the default key to be unavailable

This solves the issue where no default key can be added after -salvagewallet.

360cfe142c

BitcoinPullTester commented at 7:04 PM on April 26, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/360cfe142c552ac5c4d904a1e970390188151ca8 for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
laanwj commented at 6:10 AM on May 3, 2013: member

ACK
jgarzik commented at 9:13 PM on May 3, 2013: contributor

ACK
gavinandresen referenced this in commit 61a845dcb6 on May 3, 2013
gavinandresen merged this on May 3, 2013
gavinandresen closed this on May 3, 2013
sipa deleted the branch on May 3, 2013
laudney referenced this in commit 9e30d67568 on Mar 19, 2014
webwakko commented at 10:25 AM on December 7, 2017: none

Hi, Can someone tell me how to apply this fix for the ecrypted wallet and the use of -salvagewallet? It seems the links a broken and I'm kinda new to this, but I have a wallet that has an urgent need to be fixed .. talking about a lot of $'s :s
achow101 commented at 4:44 PM on December 7, 2017: member

@webwakko Please see #10952 which actually allows for the default key to not be in the wallet. That PR is currently merged into master. If you want to be able to use that behavior, just build master.
webwakko commented at 10:27 PM on December 7, 2017: none

Hi mate,

Thanks for getting back to me on this one, but to be honest.. this is still a bit chinese for me... here's the problem :

I have an alt coin wallet, that's is actualy using the core qt-wallet which has basicly the same code behind it.. just don't know which fork it is. Maybe you heard about the DeepOnion coin ? I have almost 3k of onions on it, but it cannot dump a private key for the address where these onions are located. I'm looking to find a solution since several weeks now but without luck... So... It would be great if you could assist me in trying to recover that private key so i can move them to a wallet that actualy has a private key (which I already created and own) I'm willing to reward you for this... current value for 1 onion is about $3 or even a bit more... If you can help me fix this problem by some simple guidance.. I'll send you a very nice reward for this. You can count on it.

Let me know if you can assist me in this one.

With friendly regards,

Webwakko

2017-12-07 17:45 GMT+01:00 Andrew Chow notifications@github.com:

@webwakko https://github.com/webwakko Please see #10952 https://github.com/bitcoin/bitcoin/pull/10952 which actually allows for the default key to not be in the wallet. That PR is currently merged into master. If you want to be able to use that behavior, just build master.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/bitcoin/bitcoin/pull/2566#issuecomment-350025179, or mute the thread https://github.com/notifications/unsubscribe-auth/AGjFNdjcmsJRTaOCyjPc6k2cgySEwTa9ks5s-BY9gaJpZM4Am9M5 .
DrahtBot locked this on Sep 8, 2021

Contributors

Milestone
v0.8.2