check if block is known before re-importing (#161)

substrate/substrate/client/src/client.rs

@@ -289,7 +289,6 @@ impl<B, E> Client<B, E> where
 		body: Option<block::Body>,
 	) -> error::Result<ImportResult> {
 		// TODO: import lock
-		// TODO: validate block
 		// TODO: import justification.
 		let (header, justification) = header.into_inner();
 		match self.backend.blockchain().status(BlockId::Hash(header.parent_hash))? {

substrate/substrate/network/src/sync.rs

@@ -230,6 +230,18 @@ impl ChainSync {
 					let hash = header_hash(&header);
 					let parent = header.parent_hash;
 					let is_best = best_seen.as_ref().map_or(false, |n| number >= *n);
+
+					// check whether the block is known before importing.
+					match protocol.chain().block_status(&BlockId::Hash(hash)) {
+						Ok(BlockStatus::InChain) => continue,
+						Ok(_) => {},
+						Err(e) => {
+							debug!(target: "sync", "Error importing block {}: {:?}: {:?}", number, hash, e);
+							self.restart(io, protocol);
+							return;
+						}
+					}
+
 					let result = protocol.chain().import(
 						is_best,
 						header,