Handling empty response TryReadStringArrayWithLengthHeader

libs/client/GarnetClientProcessReplies.cs

@@ -54,7 +54,7 @@ unsafe bool ProcessReplyAsString(ref byte* ptr, byte* end, out string result, ou
                     if (!RespReadResponseUtils.TryReadStringArrayWithLengthHeader(out var resultArray, ref ptr, end))
                         return false;
                     // Return first element of array
-                    if (resultArray != null) result = resultArray[0];
+                    if (resultArray != null && resultArray.Length > 0) result = resultArray[0];
                     break;
 
                 default:

test/Garnet.test/GarnetClientTests.cs

@@ -354,6 +354,22 @@ public async Task CanUseSetNxStringResultAsync()
             ClassicAssert.AreEqual("Hello", resultMykey);
         }
 
+        [Test]
+        public async Task ShouldNotThrowExceptionForEmptyArrayResponseAsync()
+        {
+            using var server = TestUtils.CreateGarnetServer(TestUtils.MethodTestDir);
+            server.Start();
+
+            using var db = TestUtils.GetGarnetClient();
+            await db.ConnectAsync();
+
+            ClassicAssert.DoesNotThrowAsync(async () =>
+            {
+                var result = await db.ExecuteForStringResultAsync("KEYS", ["*"]);
+                ClassicAssert.IsNull(result);
+            });
+        }
+
         [Test]
         public async Task CanUseMGetTests([Values] bool disableObjectStore)
         {