Disallow NaN intervals in LLTimers

Author: HaroldCindy

VM/src/llltimers.cpp

@@ -126,8 +126,8 @@ static int lltimers_on(lua_State *L)
         luaL_typeerror(L, 3, "function or callable table");
     lua_settop(L, 3);
 
-    if (seconds < 0.0)
-        luaL_errorL(L, "timer interval must be positive or 0");
+    if (seconds < 0.0 || std::isnan(seconds))
+        luaL_errorL(L, "timer interval must be a positive number or 0");
 
     // Get current time
     double current_time = sl_state->clockProvider ? sl_state->clockProvider(L) : 0.0;
@@ -182,8 +182,8 @@ static int lltimers_once(lua_State *L)
         luaL_typeerror(L, 3, "function or callable table");
     lua_settop(L, 3);
 
-    if (seconds < 0.0)
-        luaL_errorL(L, "timer interval must be positive or 0");
+    if (seconds < 0.0 || std::isnan(seconds))
+        luaL_errorL(L, "timer interval must be a positive number or 0");
 
     // Get current time
     double current_time = sl_state->clockProvider ? sl_state->clockProvider(L) : 0.0;

tests/conformance/lltimers.lua

@@ -135,6 +135,10 @@ local success, err = pcall(function()
 end)
 assert(success == false)
 
+-- Test NaN interval
+assert_errors(function() LLTimers:every(0/0, function() end) end, "timer interval must be a positive number or 0")
+assert_errors(function() LLTimers:once(0/0, function() end) end, "timer interval must be a positive number or 0")
+
 -- Test zero interval (0 means "ASAP" and is valid)
 timer1_count = 0
 local function zero_interval_handler()