Skip to content

Conversation

@blondfrogs
Copy link
Contributor

@blondfrogs blondfrogs commented Aug 17, 2018

Problem:

When a user would shutdown the wallet, after they issued an asset, and transfer an asset within a couple blocks then performed a wallet shutdown.

When the wallet is loaded back up, it would undo a certain amount blocks to verify the validity of the most recent chain. This allowed the database and memory to become out on sync because it was Writing the transfer asset data before the issued asset data.

Fixed - This change fixes the problem by changing the order the assets database performs its Write/Erases, so they overlap in the correct order.
Fixed - Fixed units tests that broke when #223 was implemented

Copy link
Contributor

@cfox cfox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@cfrogjump cfrogjump merged commit b265da9 into release_2.0.4 Aug 17, 2018
@blondfrogs blondfrogs deleted the database_bug_fix_2_0_4 branch August 17, 2018 15:37
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.

4 participants