fix ecrecover pass invalid params lead to core

JimmyShi22 · Jan 10, 2024 · dbe78bb · dbe78bb
1 parent f3922dd
commit dbe78bb
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/bcos-executor/src/vm/HostContext.cpp b/bcos-executor/src/vm/HostContext.cpp
@@ -288,11 +288,18 @@ evmc_result HostContext::callBuiltInPrecompiled(
         {
             callResults->gas = _request->gas - gasUsed;
         }
-        auto [success, output] =
-            m_executive->executeOriginPrecompiled(_request->receiveAddress, ref(_request->data));
-        resultCode =
-            (int32_t)(success ? TransactionStatus::None : TransactionStatus::RevertInstruction);
-        resultData.swap(output);
+        try
+        {
+            auto [success, output] = m_executive->executeOriginPrecompiled(
+                _request->receiveAddress, ref(_request->data));
+            resultCode =
+                (int32_t)(success ? TransactionStatus::None : TransactionStatus::RevertInstruction);
+            resultData.swap(output);
+        }
+        catch (...)
+        {
+            resultCode = (int32_t)TransactionStatus::RevertInstruction;
+        }
     }
     else
     {