dmigwi/migration-test

[BTC] : crash when syncing watch only wallet

Opened this issue · 5 comments

dmigwi commented

i got a crash when syncing my watch only wallet, this wallet has been previously synced in the past without any issues, i decided to enable sync for the wallet today and this crash occurred

panic: unable to synchronize wallet to chain: unable to perform wallet recovery: account 0 not found

goroutine 294 [running]:
github.com/btcsuite/btcwallet/wallet.(*Wallet).handleChainNotifications(0xc000c36c60)
 /home/kennedy/Projects/pkg/mod/github.com/btcsuite/btcwallet@v0.16.5/wallet/chainntfns.go:117 +0x1048
created by github.com/btcsuite/btcwallet/wallet.(*Wallet).SynchronizeRPC
 /home/kennedy/Projects/pkg/mod/github.com/btcsuite/btcwallet@v0.16.5/wallet/wallet.go:213 +0x245
exit status 2

image

(Issue originally authored by: @dmigwi in gitlab.com)

dmigwi commented

@dreacot how did you trigger this crash? Please advice on how I can replicate the same on my end

I have tried to use public keys from two different accounts but the same BTC wallet and they work well:

  • Account 1 : vpub5YiGSjdqe454xHenmaqxcpWAoyRupZWdvWPesmAMKjJv612KVaMKVn3w4RScshzLGtmnswgNoWVGJtpgnmPyxq6k5miMdm2N2zjUJKVzPxU
  • Account 2: vpub5YiGSjdqe454yR354MWkAjvevgDmxnnE7dmBFR5jz6Hs6eKCfiNSzWjf4g4DAaLkaiYdUTaHdCkJGz1eWft7mJiWhYfPzwQgM5eDS6rgBM9

The two accounts sync just as expected.

I have also tried to alter a valid BTC public key input, but that was easily caught by the validators already in place.

image

(Comment originally authored by: @dmigwi in gitlab.com)

dmigwi commented

Issue finally resolved!

(Comment originally authored by: @dmigwi in gitlab.com)

dmigwi commented

This is another test!

(Comment originally authored by: @dmigwi in gitlab.com)

dmigwi commented

could the test be longer?

(Comment originally authored by: @dmigwi in gitlab.com)

dmigwi commented

Hello

(Comment originally authored by: @dmigwi in gitlab.com)