fix: hardcode max vm cycles in keeper

gno.land/pkg/sdk/vm/keeper.go

@@ -22,6 +22,11 @@
 const (
 	maxAllocTx    = 500 * 1000 * 1000
 	maxAllocQuery = 1500 * 1000 * 1000 // higher limit for queries
+
+	// maxVMCycles is the maximum number of cycles allowed while executing a single VM
+	// message. Ideally this should not be needed, as execution should halt when out of
+	// gas. The worst case scenario is that this value is used as a fallback.
+	maxVMCycles = 10_000_000
 )
 
 // vm.VMKeeperI defines a module interface that supports Gno
@@ -57,14 +62,19 @@
 	stdlibsDir string,
 	maxCycles int64,
 ) *VMKeeper {
+	cycles := maxCycles
+	if cycles <= 0 || cycles > maxVMCycles {
+		cycles = maxVMCycles
+	}
+
 	// TODO: create an Options struct to avoid too many constructor parameters
 	vmk := &VMKeeper{
 		baseKey:    baseKey,
 		iavlKey:    iavlKey,
 		acck:       acck,
 		bank:       bank,
 		stdlibsDir: stdlibsDir,
-		maxCycles:  maxCycles,
+		maxCycles:  cycles,
 	}
 	return vmk
 }