fix(l2geth): Log 'end of OVM execution' correctly (#1080)

d9fd67d2 · Maurelian · GitHub · f409ce75 · d9fd67d2 · d9fd67d2
Commit d9fd67d2 authored Jun 14, 2021 by Maurelian Committed by GitHub Jun 14, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 4 deletions

ten-pumas-perform.md .changeset/ten-pumas-perform.md +5 -0

evm.go l2geth/core/vm/evm.go +3 -4

No files found.
--- a/.changeset/ten-pumas-perform.md
+++ b/.changeset/ten-pumas-perform.md
+---
+'@eth-optimism/l2geth': patch
+---
+
+Correctly log 'end of OVM execution' message.
--- a/l2geth/core/vm/evm.go
+++ b/l2geth/core/vm/evm.go
@@ -464,10 +464,9 @@ func (evm *EVM) Call(caller ContractRef, addr common.Address, input []byte, gas
 					ret = []byte{}
 				}
 			}
-		}
-
-		if evm.Context.EthCallSender == nil {
-			log.Debug("Reached the end of an OVM execution", "ID", evm.Id, "Return Data", hexutil.Encode(ret), "Error", err)
+			if evm.Context.EthCallSender == nil {
+				log.Debug("Reached the end of an OVM execution", "ID", evm.Id, "Return Data", hexutil.Encode(ret), "Error", err)
+			}
 		}
 	}