diff options
| author | Sonic <sonic@dexon.org> | 2018-12-27 16:25:45 +0800 |
|---|---|---|
| committer | Wei-Ning Huang <w@dexon.org> | 2019-04-09 13:49:59 +0800 |
| commit | 742f8857f0073eb5458530d5433e556d7bf8ba3a (patch) | |
| tree | 106762efc68c7fa6f93094dd8def441e80bcf38a | |
| parent | 105453763f0b30769bd63b0d71e9688eded6fcc1 (diff) | |
| download | dexon-742f8857f0073eb5458530d5433e556d7bf8ba3a.tar dexon-742f8857f0073eb5458530d5433e556d7bf8ba3a.tar.gz dexon-742f8857f0073eb5458530d5433e556d7bf8ba3a.tar.bz2 dexon-742f8857f0073eb5458530d5433e556d7bf8ba3a.tar.lz dexon-742f8857f0073eb5458530d5433e556d7bf8ba3a.tar.xz dexon-742f8857f0073eb5458530d5433e556d7bf8ba3a.tar.zst dexon-742f8857f0073eb5458530d5433e556d7bf8ba3a.zip | |
dex/downloader: fix bug when syncing (#106)
Since blocks will interleave around round change, we will probably need
to verify blocks at previous round.
| -rw-r--r-- | dex/downloader/downloader.go | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/dex/downloader/downloader.go b/dex/downloader/downloader.go index 809fe7e4a..6ff8c122e 100644 --- a/dex/downloader/downloader.go +++ b/dex/downloader/downloader.go @@ -480,9 +480,9 @@ func (d *Downloader) syncWithPeer(p *peerConnection, hash common.Hash, number ui return fmt.Errorf("origin header not exists, number: %d", origin) } - // prepare state origin - 2 + // prepare state origin - 3 d.gov = newGovernance(govState) - for i := uint64(0); i < 3; i++ { + for i := uint64(0); i < 4; i++ { if originHeader.Round >= i { h := d.gov.GetRoundHeight(originHeader.Round - i) s, err := d.lightchain.GetGovStateByNumber(h) @@ -494,6 +494,17 @@ func (d *Downloader) syncWithPeer(p *peerConnection, hash common.Hash, number ui } d.verifierCache = dexCore.NewTSigVerifierCache(d.gov, 5) + + // warm up verifierCache + if originHeader.Round > 0 { + ok, err := d.verifierCache.Update(originHeader.Round - 1) + if err != nil { + return err + } + if !ok { + return fmt.Errorf("can not update verifier cache") + } + } } // Initiate the sync using a concurrent header and content retrieval algorithm |