[walletdb] don't report minversion wallet entry as unknown #13967

pull instagibbs wants to merge 1 commits into bitcoin:master from instagibbs:unknownminversion changing 1 files +2 −1

instagibbs commented at 3:35 PM on August 14, 2018: member

It is known in WalletBatch::LoadWallet
don't report minversion wallet entry as unknown 321159e53e
laanwj added the label Wallet on Aug 14, 2018
DrahtBot commented at 5:58 PM on August 14, 2018: member
Note to reviewers: This pull request conflicts with the following ones:
- #13825 ([wallet] Kill accounts by jnewbery)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
heyhpython approved
laanwj commented at 7:58 AM on August 22, 2018: member

utACK 321159e53e800c1df2d8dfd6ac03374f1829c327
laanwj merged this on Aug 22, 2018
laanwj closed this on Aug 22, 2018
laanwj referenced this in commit 0738b88fe0 on Aug 22, 2018
jnewbery commented at 2:09 PM on August 23, 2018: member

post-merge utACK 321159e53e800c1df2d8dfd6ac03374f1829c327
r8921039 commented at 8:21 PM on December 27, 2018: contributor

I just encountered this "unknown wallet records" issue today when running the latest 0.17.1 (upgraded from 0.17.0). Not sure bitcoincore's backport policy but it seems to me it would be nice to have this simple fix on 0.17.2.
Munkybooty referenced this in commit 7b5b302c57 on Jun 30, 2021
Munkybooty referenced this in commit e8cf418475 on Jun 30, 2021
Munkybooty referenced this in commit 6210e4eda6 on Jul 1, 2021
Munkybooty referenced this in commit 96eb177639 on Jul 2, 2021
Munkybooty referenced this in commit 6a5c3a78d6 on Jul 2, 2021
Munkybooty referenced this in commit 243ee3137a on Jul 2, 2021
DrahtBot locked this on Sep 8, 2021

Contributors

Labels