fix executor error handling (0xPolygonHermez#2109)

imnotsure-x · May 18, 2023 · 7f7e467 · 7f7e467
1 parent 1d29c35
commit 7f7e467
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/state/runtime/executor/errors.go b/state/runtime/executor/errors.go
@@ -121,7 +121,7 @@ func RomErr(errorCode pb.RomError) error {
 	case ROM_ERROR_EXECUTION_REVERTED:
 		return runtime.ErrExecutionReverted
 	case ROM_ERROR_OUT_OF_COUNTERS_STEP:
-		return runtime.ErrOutOfCountersKeccak
+		return runtime.ErrOutOfCountersStep
 	case ROM_ERROR_OUT_OF_COUNTERS_KECCAK:
 		return runtime.ErrOutOfCountersKeccak
 	case ROM_ERROR_OUT_OF_COUNTERS_BINARY:
@@ -183,6 +183,8 @@ func RomErrorCode(err error) pb.RomError {
 		return pb.RomError(ROM_ERROR_CONTRACT_ADDRESS_COLLISION)
 	case runtime.ErrExecutionReverted:
 		return pb.RomError(ROM_ERROR_EXECUTION_REVERTED)
+	case runtime.ErrOutOfCountersStep:
+		return pb.RomError(ROM_ERROR_OUT_OF_COUNTERS_STEP)
 	case runtime.ErrOutOfCountersKeccak:
 		return pb.RomError(ROM_ERROR_OUT_OF_COUNTERS_KECCAK)
 	case runtime.ErrOutOfCountersBinary:

diff --git a/state/runtime/runtime.go b/state/runtime/runtime.go
@@ -21,6 +21,8 @@ var (
 	ErrContractAddressCollision = errors.New("contract address collision")
 	// ErrExecutionReverted indicates the execution has been reverted
 	ErrExecutionReverted = errors.New("execution reverted")
+	// ErrOutOfCountersStep indicates there are not enough step counters to continue the execution
+	ErrOutOfCountersStep = errors.New("not enough step counters to continue the execution")
 	// ErrOutOfCountersKeccak indicates there are not enough keccak counters to continue the execution
 	ErrOutOfCountersKeccak = errors.New("not enough keccak counters to continue the execution")
 	// ErrOutOfCountersBinary indicates there are not enough binary counters to continue the execution