You should enable os.tmpname
Generate a name that can be used for a temporary file. This only generates a name, it does not open a file.

http://lua-users.org/wiki/OsLibraryTutorial
could be useful 🙄

@CrosRoad95 os.tmpname returns a string with the full directory. It might be a security concern.

os.clock returns seconds, while getTickCount() returns ms

@Pirulax it returns a float of 0.001 precision in seconds format, which is the same as milliseconds.

I dont know.
Tried to run os.clock() on repl.it, but it's wired a little bit there.
I dont have lua interpreter installed, so I cant test that.
But I remember that I've read something about os.clock() returning seconds only, and if I remembered it correctly, I've read that at math.randomseed() function on lua-users

@Pirulax it does return seconds, but in 0.001 precision. Read the example from Lua documentation: http://lua-users.org/wiki/OsLibraryTutorial

I'm just going to give my two cents here to @Dezash: wouldn't it be better to push the CLuaUtilDefs::DisabledFunction method to the Lua stack, instead of a nil value, to disable these unsafe OS library functions? That method logs an error and pushes false to the Lua stack, which states more clearly MTA's intent to disable those functions to the scripter. Just replacing them with a nil value, on the other hand, is a bit less clear at first hand.

I think (but I'm not sure, I'm not an active C++ developer who works with Lua) you can use lua_pushcfunction(m_luaVM, CLuaUtilDefs::DisabledFunction) to accomplish this, similarly to other disabled functions that follow.

@AlexTMjugador agreed. I did mention this in my review but I don't think he noticed 🙂

Good idea, I missed qaisjp's note somehow.

We might want to disable os.setlocale as well as it changes the locale globally for the entire application. This could result in unwanted side effects.