diff --git a/client/src/call_executor.rs b/client/src/call_executor.rs index a41e037636064..d37d2b0068694 100644 --- a/client/src/call_executor.rs +++ b/client/src/call_executor.rs @@ -262,9 +262,6 @@ where &self.executor, method, call_data, - // Passing `None` here, since we don't really want to prove anything - // about our local keys. - None, ) .map_err(Into::into) } diff --git a/client/src/light/call_executor.rs b/client/src/light/call_executor.rs index a4a8a549de93c..29b9ca46af1fe 100644 --- a/client/src/light/call_executor.rs +++ b/client/src/light/call_executor.rs @@ -113,7 +113,7 @@ impl CallExecutor for _manager: ExecutionManager, native_call: Option, recorder: &Option>, - extensions: Option, + extensions: Option, ) -> ClientResult> where ExecutionManager: Clone { // there's no actual way/need to specify native/wasm execution strategy on light node // => we can safely ignore passed values @@ -277,7 +277,6 @@ fn check_execution_proof_with_make_header( exec: &Exec, method: &str, call_data: &[u8], - keystore: Option, ) -> Result<(Vec, StorageProof), Box> where B: Backend, @@ -461,7 +460,7 @@ where { let trie_backend = backend.as_trie_backend() .ok_or_else(|| Box::new(ExecutionError::UnableToGenerateProof) as Box)?; - prove_execution_on_trie_backend(trie_backend, overlay, exec, method, call_data, keystore) + prove_execution_on_trie_backend(trie_backend, overlay, exec, method, call_data) } /// Prove execution using the given trie backend, overlayed changes, and call executor. @@ -479,7 +478,6 @@ pub fn prove_execution_on_trie_backend( exec: &Exec, method: &str, call_data: &[u8], - keystore: Option, ) -> Result<(Vec, StorageProof), Box> where S: trie_backend_essence::TrieBackendStorage, @@ -487,12 +485,8 @@ where Exec: CodeExecutor, { let proving_backend = proving_backend::ProvingBackend::new(trie_backend); - let mut extensions = Extensions::new(); - if let Some(keystore) = keystore { - extensions.register(keystore); - } let mut sm = StateMachine::<_, H, _, InMemoryChangesTrieStorage, Exec>::new( - &proving_backend, None, overlay, exec, method, call_data, extensions, + &proving_backend, None, overlay, exec, method, call_data, Extensions::default(), ); let (result, _, _) = sm.execute_using_consensus_failure_handler::<_, NeverNativeValue, fn() -> _>( @@ -512,7 +506,6 @@ pub fn execution_proof_check( exec: &Exec, method: &str, call_data: &[u8], - keystore: Option, ) -> Result, Box> where H: Hasher, @@ -520,7 +513,7 @@ where H::Out: Ord + 'static, { let trie_backend = create_proof_check_backend::(root.into(), proof)?; - execution_proof_check_on_trie_backend(&trie_backend, overlay, exec, method, call_data, keystore) + execution_proof_check_on_trie_backend(&trie_backend, overlay, exec, method, call_data) } /// Check execution proof on proving backend, generated by `prove_execution` call. @@ -530,18 +523,13 @@ pub fn execution_proof_check_on_trie_backend( exec: &Exec, method: &str, call_data: &[u8], - keystore: Option, ) -> Result, Box> where H: Hasher, Exec: CodeExecutor, { - let mut extensions = Extensions::new(); - if let Some(keystore) = keystore { - extensions.register(keystore); - } let mut sm = StateMachine::<_, H, _, InMemoryChangesTrieStorage, Exec>::new( - trie_backend, None, overlay, exec, method, call_data, extensions, + trie_backend, None, overlay, exec, method, call_data, Extensions::default(), ); sm.execute_using_consensus_failure_handler::<_, NeverNativeValue, fn() -> _>( @@ -914,7 +902,6 @@ mod tests { &executor, "test", &[], - None, ).unwrap(); // check proof locally @@ -925,7 +912,6 @@ mod tests { &executor, "test", &[], - None, ).unwrap(); // check that both results are correct