Fix bug with Lua infallible allocations (#1139)

* if here is wrong, was pointlessly recreating luarunners

* add a test that would have caught this mistake

Author: kevin-montrose

libs/server/Lua/LuaStateWrapper.cs

@@ -766,7 +766,7 @@ private bool TryExitInfallibleAllocationRegion()
 
             var ret = customAllocator.TryExitInfallibleAllocationRegion();
 
-            if (ret)
+            if (!ret)
             {
                 NeedsDispose = true;
             }

test/Garnet.test/LuaScriptRunnerTests.cs

@@ -737,6 +737,30 @@ public void InternalFunctionsIgnoredInAllowedFunctions()
             ClassicAssert.AreEqual("nil", res);
         }
 
+        [Test]
+        public void NeedsDisposeCheck()
+        {
+            foreach (var mode in Enum.GetValues<LuaMemoryManagementMode>())
+            {
+                foreach (var limit in new[] { null, "1m" })
+                {
+                    if (limit != null && mode == LuaMemoryManagementMode.Native)
+                    {
+                        continue;
+                    }
+
+                    var opts = new LuaOptions(mode, limit, Timeout.InfiniteTimeSpan, LuaLoggingMode.Silent, []);
+
+                    using var runner = new LuaRunner(opts, "return 1");
+
+                    runner.CompileForRunner();
+                    _ = runner.RunForRunner([], []);
+
+                    ClassicAssert.IsFalse(runner.NeedsDispose);
+                }
+            }
+        }
+
         private sealed class FakeLogger : ILogger, IDisposable
         {
             private readonly List<string> logs = new();