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

Fix updating the UTXO set for imported addresses #133

Merged
merged 1 commit into from
Mar 28, 2016

Conversation

cjepson
Copy link
Contributor

@cjepson cjepson commented Mar 28, 2016

Wallet would fail to update UTXOs for imported address outputs while
rescanning after using importprivkey or importscript if the transaction
already existed in the database. The new code is rearranged to always
update spending data even if the transaction already exists.

Fixes #132.

@jcvernaleo
Copy link
Member

Looks good on siment and testnet.
tACK

@jcvernaleo
Copy link
Member

It does need a squash of course.

@cjepson cjepson force-pushed the cj_280316importedtx branch from 9f858bd to 4a113dc Compare March 28, 2016 17:34
@alexlyp
Copy link
Member

alexlyp commented Mar 28, 2016

tACK

Wallet would fail to update UTXOs for imported address outputs while
rescanning after using importprivkey or importscript if the transaction
already existed in the database. The new code is rearranged to always
update spending data even if the transaction already exists.

Fixes btcsuite#132.
@cjepson cjepson force-pushed the cj_280316importedtx branch from 4a113dc to a488b38 Compare March 28, 2016 17:48
@jcvernaleo jcvernaleo merged commit a488b38 into decred:master Mar 28, 2016
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.

3 participants