Post
Topic
Board Announcements (Altcoins)
Re: [ANN] CommunityCoin (COMM) Fair & FREE Distribution | Pure PoS | Join Community
by
mamamia2014
on 19/04/2014, 14:32:43 UTC
Hello Everyone,


I just commited some changes into github to block transactions from the lost wallet. I tried it both in an existing installation and in a fresh one (which requires the blockchain to be downloaded) and it seems to be working ok.

I'd like to have it peer-reviewed and tested by some of you before we build the official wallets.


Thanks,


Adriano

I'm sure you will have thought of this already, but can you confirm that transactions are still blocked if the private key is exported and then imported to a new wallet?

Yep, the patch is watching two places:

1. New transactions, not yet in a block. Do not accept any new transaction spent coins tied to that address into mempool (this prevents such transaction being add in block generated by authentic wallets);

2. When receiving a new block, it checks if there are any transactions sent from that address in blocks higher than 28647 (this is the last block containing a valid transaction on the wallet before its key was lost.


Adriano

Sounds promising.

Do you need help for testing?

If you build your own wallet you can already start using it (backup your wallet before just in case), nothing is supposed to change, then check your balance, send and receive some transaction. This would test it for existing installations where blockchain is already downloaded.

Another test would be deleting the blockchain and starting the wallet (again, don't forget to backup wallet). Then the same tests as before (balance, sending and receiving)

For those who use the  windows, linux or osx wallets we will need to wait until they are updated.

Adriano
Hi, Adriano,

Has the dev compiled and mergered the modified code yet?
Will we have a new wallet,right?

I haven't heard from him yet.
I'm no expert on github, but it seems the code can be viewed and downloaded from the pull request even before it is accepted and merged.

You are right about having a new wallet version.


Adriano

Thank you for your explanation. And waiting for the new wallet. Hope it can solve the problem.