[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: Userland Lua - concerns with require
On Thu, Oct 28, 2010 at 09:21:36AM +0200, Marc Balmer wrote:
> Am 27.10.10 23:37, schrieb Phil Nelson:
> > On Wednesday 27 October 2010 2:21:34 pm Marc Balmer wrote:
> >> I will change the default path specifications.
> > What specific change? Removing . or moving it to the end?
> > I'd argue that removing . is the correct thing to do, not
> > just moving it to the end if you are worried about security.
> > But from my knowledge of lua it expects . in the path.
> > Your two previous posts didn't convey the details of the solution.
> After considerations, email exchange with the makers of Lua, and testing
> realword systems I think probably the best thing to do is to move the
> '.' parts to the end of the list:
> The obvious security risk is that someone puts a Lua file in your
> homedirectory with the name of an _existing_ module, e.g. some database
> adapter, and tricks you into running this code instead.
> When '.' is moved to the end of the list, this kind of exploit is no
> longer possible. This is also what the makers of Lua did for the next
> release, Lua 5.2.
> I am a bit hesitant right now to remove the '.' entries completely, I am
> still trying to assess the consequences. That said, I am open for (well
> reasoned) opinions.
So you still plan to import it with having '.' in the path which means we
will now get something new in the base system which we know that it could
cause security issues.
Main Index |
Thread Index |