fix: don't record the call error twice if we hit the max depth (#1953)

We'll record the trace error in `call_actor` itself.
filecoin-project · Dec 15, 2023 · 40bdf9c · 40bdf9c
1 parent a8f9be1
commit 40bdf9c
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/fvm/src/call_manager/default.rs b/fvm/src/call_manager/default.rs
@@ -919,11 +919,9 @@ where
         F: FnOnce(&mut Self) -> Result<V>,
     {
         if self.call_stack_depth >= self.machine.context().max_call_depth {
-            let sys_err = syscall_error!(LimitExceeded, "message execution exceeds call depth");
-            if self.machine.context().tracing {
-                self.trace(ExecutionEvent::CallError(sys_err.clone()));
-            }
-            return Err(sys_err.into());
+            return Err(
+                syscall_error!(LimitExceeded, "message execution exceeds call depth").into(),
+            );
         }
 
         self.call_stack_depth += 1;