Hi,

sorry to bother you again. I have another request probably also into the 
direction of libraries. In order to make the script work with all engines, we 
need to execute the script (instead of loading it with require() as this would 
only work in lualatex). As a result, the script will be totally unrestricted 
and none of the security wrappers that would be in place with shell-restricted 
enabled are available. The issue is we do still want to make the script be 
secure (actually we want it to be in the list of commands allowed with 
shell-restricted in the end).
For this we already wrap security sensitive functions like io.lines to put the 
security guards of shell-restricted back in place. But here it is easy to miss 
some checks and also it weakens security in general if we need to implement 
these security wrappers over and over again (maybe this can be compared to 
"never roll your own crypto"). Also in the future we might find a bug / 
vulnerability in the security wrapper and it would be not only laborious but 
also prone to errors (keeping attack vectors open) having to apply the patch to 
all utilities shipping their own security wrappers.

So you might already guessed what I'm asking for: Can we somehow make the 
wrappers that are defined in luatex anyhow (and it wouldn't make much sense to 
implement them anywhere else) available to texlua scripts? Precisely, I'm 
speaking of the functions defined / assigned here [1].

You'd probably just need to move these functions to a security/io library and 
refer to it at the place where you currently define these wrappers. Oh and of 
course the table reflecting this library probably should be read-only in order 
to avoid the user tampering (deleting or even worse modifying) these wrappers. 
But this can be done in lua e.g. like it is documented in PIL [2] (I know this 
is the old 5.0 version, but the snippet still works reliably).
You could of course still make copies of the functions for the luatex-core.lua 
file, but having the library read-only probably also makes sense in this regard.

I see you probably want to keep luatex small in order to make it easier to 
maintain and move as much as possible (like the pathutil in my previous 
request) to external lua libraries. But in this case any duplication of this 
code makes it more likely someone forgets to patch it or makes a mistake in 
implementing it correctly.

If you consider implementing this and I can help somehow, just let me know.

With kind regards
Lukas

[1]: 
https://gitlab.lisn.upsaclay.fr/texlive/luatex/-/blob/07c0b354d913a32dd3faf79f05f0f103c55416d0/source/texk/web2c/luatexdir/lua/luatex-core.lua#L322-L341
[2]: https://www.lua.org/pil/13.4.5.html

Reply via email to