503f591e88
This change modifies the order in which transaction to watched addresses are processed and when frontend notifications occur. Due to btcd notifying all transactions before sending the blockconnected notification, the UTXO and transaction stores can be modified without sending any frontend notifications, and then a single frontend notification is sent when the blockconnected notification arrives. The order in which each file is synced to disk was also changed to write out the UTXO and transaction stores before writing the wallet. This is to prevent a race where wallet closes after writing the dirty wallet, but before the dirty UTXO store is written. In this situation, newly added UTXOs will be missed and not found again on the next wallet open during the rescan. Writing the wallet (which holds the synced-to-block information) last prevents this. An issue where the unconfirmed change UTXO created from a new transaction never being properly notified to frontends is fixed now as well.
152 lines
4.1 KiB
Go
152 lines
4.1 KiB
Go
/*
|
|
* Copyright (c) 2013 Conformal Systems LLC <info@conformal.com>
|
|
*
|
|
* Permission to use, copy, modify, and distribute this software for any
|
|
* purpose with or without fee is hereby granted, provided that the above
|
|
* copyright notice and this permission notice appear in all copies.
|
|
*
|
|
* THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
|
|
* WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
|
|
* MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
|
|
* ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
|
|
* WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
|
|
* ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
|
|
* OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
|
|
*/
|
|
|
|
package main
|
|
|
|
import (
|
|
"fmt"
|
|
"os"
|
|
"path/filepath"
|
|
"sync"
|
|
"time"
|
|
)
|
|
|
|
var (
|
|
// dirtyWallets holds a set of wallets that include dirty components.
|
|
dirtyWallets = struct {
|
|
sync.Mutex
|
|
m map[*BtcWallet]bool
|
|
}{
|
|
m: make(map[*BtcWallet]bool),
|
|
}
|
|
)
|
|
|
|
// DirtyWalletSyncer synces dirty wallets for cases where the updated
|
|
// information was not required to be immediately written to disk. Wallets
|
|
// may be added to dirtyWallets and will be checked and processed every 10
|
|
// seconds by this function.
|
|
//
|
|
// This never returns and is meant to be called from a goroutine.
|
|
func DirtyWalletSyncer() {
|
|
ticker := time.Tick(10 * time.Second)
|
|
for {
|
|
select {
|
|
case <-ticker:
|
|
dirtyWallets.Lock()
|
|
for w := range dirtyWallets.m {
|
|
log.Debugf("Syncing wallet '%v' to disk",
|
|
w.Wallet.Name())
|
|
if err := w.writeDirtyToDisk(); err != nil {
|
|
log.Errorf("cannot sync dirty wallet: %v",
|
|
err)
|
|
} else {
|
|
delete(dirtyWallets.m, w)
|
|
}
|
|
}
|
|
dirtyWallets.Unlock()
|
|
}
|
|
}
|
|
}
|
|
|
|
// writeDirtyToDisk checks for the dirty flag on an account's wallet,
|
|
// txstore, and utxostore, writing them to disk if any are dirty.
|
|
func (w *BtcWallet) writeDirtyToDisk() error {
|
|
// Temporary files append the current time to the normal file name.
|
|
// In caes of failure, the most recent temporary file can be inspected
|
|
// for validity, and moved to replace the main file.
|
|
timeStr := fmt.Sprintf("%v", time.Now().Unix())
|
|
|
|
wdir := walletdir(cfg, w.name)
|
|
wfilepath := filepath.Join(wdir, "wallet.bin")
|
|
txfilepath := filepath.Join(wdir, "tx.bin")
|
|
utxofilepath := filepath.Join(wdir, "utxo.bin")
|
|
|
|
// UTXOs and transactions are synced to disk first. This prevents
|
|
// any races from saving a wallet marked to be synced with block N
|
|
// and btcwallet closing while the UTXO and Tx files are only synced
|
|
// with block N-1.
|
|
|
|
// UTXOs
|
|
if w.UtxoStore.dirty {
|
|
w.UtxoStore.Lock()
|
|
defer w.UtxoStore.Unlock()
|
|
tmpfilepath := utxofilepath + "-" + timeStr
|
|
tmpfile, err := os.Create(tmpfilepath)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if _, err = w.UtxoStore.s.WriteTo(tmpfile); err != nil {
|
|
return err
|
|
}
|
|
tmpfile.Close()
|
|
|
|
// TODO(jrick): this should be atomic on *nix, but is not on
|
|
// Windows. Use _windows.go to provide atomic renames.
|
|
if err = os.Rename(tmpfilepath, utxofilepath); err != nil {
|
|
return err
|
|
}
|
|
|
|
w.UtxoStore.dirty = false
|
|
}
|
|
|
|
// Transactions
|
|
if w.TxStore.dirty {
|
|
w.TxStore.Lock()
|
|
defer w.TxStore.Unlock()
|
|
tmpfilepath := txfilepath + "-" + timeStr
|
|
tmpfile, err := os.Create(tmpfilepath)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if _, err = w.TxStore.s.WriteTo(tmpfile); err != nil {
|
|
return err
|
|
}
|
|
tmpfile.Close()
|
|
|
|
// TODO(jrick): this should be atomic on *nix, but is not on
|
|
// Windows. Use _windows.go to provide atomic renames.
|
|
if err = os.Rename(tmpfilepath, txfilepath); err != nil {
|
|
return err
|
|
}
|
|
|
|
w.TxStore.dirty = false
|
|
}
|
|
|
|
// Wallet
|
|
if w.dirty {
|
|
w.mtx.Lock()
|
|
defer w.mtx.Unlock()
|
|
tmpfilepath := wfilepath + "-" + timeStr
|
|
tmpfile, err := os.Create(tmpfilepath)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if _, err = w.WriteTo(tmpfile); err != nil {
|
|
return err
|
|
}
|
|
tmpfile.Close()
|
|
|
|
// TODO(jrick): this should be atomic on *nix, but is not on
|
|
// Windows. Use _windows.go to provide atomic renames.
|
|
if err = os.Rename(tmpfilepath, wfilepath); err != nil {
|
|
return err
|
|
}
|
|
|
|
w.dirty = false
|
|
}
|
|
|
|
return nil
|
|
}
|