Phoenix Protocol: do not execute if major syncing

Author: SurfingNerd

crates/ethcore/src/engines/hbbft/hbbft_engine.rs

@@ -498,7 +498,7 @@ impl HoneyBadgerBFT {
     fn handle_phoenix_recovery_protocol(&self) {
         if let Some(client) = self.client_arc() {
             // Skip if still syncing.
-            if self.is_syncing(&client) {
+            if self.is_major_syncing(&client) {
                 return;
             }
 
@@ -1405,6 +1405,14 @@ impl HoneyBadgerBFT {
         }
     }
 
+    fn is_major_syncing(&self, client: &Arc<dyn EngineClient>) -> bool {
+        match client.as_full_client() {
+            Some(full_client) => full_client.is_major_syncing(),
+            // We only support full clients at this point.
+            None => true,
+        }
+    }
+
     /** returns if the signer of hbbft is tracked as available in the hbbft contracts..*/
     pub fn is_available(&self) -> bool {
         self.hbbft_engine_cache.lock().is_available()