execution retval length check (#4802)

envestcc · web-flow · commit 07633c8fe60a · 2026-03-17T15:00:14.000+08:00
diff --git a/action/protocol/execution/evm/evm.go b/action/protocol/execution/evm/evm.go
@@ -371,7 +371,7 @@ func ExecuteContract(
 	}
 	stateDB.clear()
 
-	if ps.featureCtx.SetRevertMessageToReceipt && receipt.Status == uint64(iotextypes.ReceiptStatus_ErrExecutionReverted) && retval != nil && bytes.Equal(retval[:4], _revertSelector) {
+	if ps.featureCtx.SetRevertMessageToReceipt && receipt.Status == uint64(iotextypes.ReceiptStatus_ErrExecutionReverted) && len(retval) >= 4 && bytes.Equal(retval[:4], _revertSelector) {
 		// in case of the execution revert error, parse the retVal and add to receipt
 		receipt.SetExecutionRevertMsg(ExtractRevertMessage(retval))
 	}

Original file line number	Diff line number	Diff line change
`@@ -371,7 +371,7 @@ func ExecuteContract(`
`371`	`371`	`}`
`372`	`372`	`stateDB.clear()`
`373`	`373`
`374`		`- if ps.featureCtx.SetRevertMessageToReceipt && receipt.Status == uint64(iotextypes.ReceiptStatus_ErrExecutionReverted) && retval != nil && bytes.Equal(retval[:4], _revertSelector) {`
	`374`	`+ if ps.featureCtx.SetRevertMessageToReceipt && receipt.Status == uint64(iotextypes.ReceiptStatus_ErrExecutionReverted) && len(retval) >= 4 && bytes.Equal(retval[:4], _revertSelector) {`
`375`	`375`	`// in case of the execution revert error, parse the retVal and add to receipt`
`376`	`376`	`receipt.SetExecutionRevertMsg(ExtractRevertMessage(retval))`
`377`	`377`	`}`