From a7bb33e5dba4f4b24a848c64954bf1659583e5a4 Mon Sep 17 00:00:00 2001
From: Sonic <sonic@dexon.org>
Date: Thu, 27 Dec 2018 16:25:45 +0800
Subject: dex/downloader: fix bug when syncing (#106)

Since blocks will interleave around round change, we will probably need
to verify blocks at previous round.
---
 dex/downloader/downloader.go | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

(limited to 'dex')

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
-- 
cgit v1.2.3