core: fixed unwinding bad hash (#3347)

Fixed unwinding of bad hashes when already on the canon chain

core/blockchain.go

@@ -152,11 +152,16 @@ func NewBlockChain(chainDb ethdb.Database, config *params.ChainConfig, pow pow.P
 	// Check the current state of the block hashes and make sure that we do not have any of the bad blocks in our chain
 	for hash, _ := range BadHashes {
 		if header := bc.GetHeaderByHash(hash); header != nil {
+			// get the canonical block corresponding to the offending header's number
+			headerByNumber := bc.GetHeaderByNumber(header.Number.Uint64())
+			// make sure the headerByNumber (if present) is in our current canonical chain
+			if headerByNumber != nil && headerByNumber.Hash() == header.Hash() {
 				glog.V(logger.Error).Infof("Found bad hash, rewinding chain to block #%d [%x…]", header.Number, header.ParentHash[:4])
 				bc.SetHead(header.Number.Uint64() - 1)
 				glog.V(logger.Error).Infoln("Chain rewind was successful, resuming normal operation")
 			}
 		}
+	}
 	// Take ownership of this particular state
 	go bc.update()
 	return bc, nil