Merged in fix-merge-error (pull request #1240)

Fix errors caused by auto merge. Approved-by: Zhiwei <[email protected]>
citahub · Aug 22, 2018 · 2a537ba · 2a537ba
2 parents 7ff4a13 + f5b2343
commit 2a537ba
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/cita-executor/core/src/contracts/sys_config.rs b/cita-executor/core/src/contracts/sys_config.rs
@@ -126,22 +126,22 @@ impl<'a> SysConfig<'a> {
                 &[ParamType::Bool],
                 FEE_BACK_PLATFORM_CHECK.as_slice(),
                 Some(BlockId::Latest),
-            ).remove(0)
-                .to_bool()
+            ).ok()
+                .and_then(|mut x| x.remove(0).to_bool())
                 .unwrap_or_else(|| false);
         debug!("check fee back platform: {:?}", check);
         check
     }
 
-    // The owner of current chain
+    /// The owner of current chain
     pub fn chain_owner(&self) -> Address {
         let chain_owner =
             self.get_value(
                 &[ParamType::Address],
                 CHAIN_OWNER.as_slice(),
                 Some(BlockId::Latest),
-            ).remove(0)
-                .to_address()
+            ).ok()
+                .and_then(|mut x| x.remove(0).to_address())
                 .unwrap_or_else(|| [0u8; 20]);
         debug!("Get chain owner: {:?}", chain_owner);
         Address::from(chain_owner)

diff --git a/cita-executor/core/src/tests/amend_data_test.rs b/cita-executor/core/src/tests/amend_data_test.rs
@@ -35,6 +35,8 @@ fn call_vm(
         &native_factory,
         false,
         EconomicalModel::Quota,
+        false,
+        Address::from(0),
     );
     let mut out = vec![];
     ex.call(