Page MenuHomePhabricator

Fix the deserialization of CoinEntry
ClosedPublic

Authored by Fabien on Fri, Feb 5, 21:14.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABC05d6cf91570b: Fix the deserialization of CoinEntry
Summary

D9169 introduced new methods to make serialization/deserialization
easier to read and write, however the migration of CoinEntry is missing a VARINT() declaration and triggers a database corruption.

Test Plan
ninja all check-all

./src/bitcoind

Should return no database corruption.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien requested review of this revision.Fri, Feb 5, 21:14
Fabien planned changes to this revision.Fri, Feb 5, 21:19
Fabien edited the summary of this revision. (Show Details)

Actually fix the issue

majcosta added a subscriber: majcosta.

tested locally, looks good

This revision is now accepted and ready to land.Fri, Feb 5, 21:46
This revision was automatically updated to reflect the committed changes.