Check number of keys in EVAL/EVALSHA (#1067)

Return correct type in SCRIPT EXISTS

Co-authored-by: prvyk <[email protected]>

libs/server/Lua/LuaCommands.cs

@@ -2,10 +2,8 @@
 // Licensed under the MIT license.
 
 using System;
-using System.Collections.Generic;
 using Garnet.common;
 using Microsoft.Extensions.Logging;
-using Tsavorite.core;
 
 namespace Garnet.server
 {
@@ -28,6 +26,11 @@ private unsafe bool TryEVALSHA()
                 return AbortWithWrongNumberOfArguments("EVALSHA");
             }
 
+            if (!parseState.TryGetInt(1, out var n) || (n < 0) || (n > count - 2))
+            {
+                return AbortWithErrorMessage(CmdStrings.RESP_ERR_GENERIC_VALUE_IS_NOT_INTEGER);
+            }
+
             ref var digest = ref parseState.GetArgSliceByRef(0);
 
             var convertedToLower = false;
@@ -99,6 +102,11 @@ private unsafe bool TryEVAL()
                 return AbortWithWrongNumberOfArguments("EVAL");
             }
 
+            if (!parseState.TryGetInt(1, out var n) || (n < 0) || (n > count - 2))
+            {
+                return AbortWithErrorMessage(CmdStrings.RESP_ERR_GENERIC_VALUE_IS_NOT_INTEGER);
+            }
+
             ref var script = ref parseState.GetArgSliceByRef(0);
 
             // that this is stack allocated is load bearing - if it moves, things will break
@@ -162,7 +170,7 @@ private bool NetworkScriptExists()
                     exists = storeWrapper.storeScriptCache.ContainsKey(sha1Arg) ? 1 : 0;
                 }
 
-                while (!RespWriteUtils.TryWriteArrayItem(exists, ref dcurr, dend))
+                while (!RespWriteUtils.TryWriteInt32(exists, ref dcurr, dend))
                     SendAndReset();
             }