luci-base: sys: prevent path traversal via sys.init routines
authorJo-Philipp Wich <jo@mein.io>
Wed, 19 Jan 2022 15:32:52 +0000 (16:32 +0100)
committerJo-Philipp Wich <jo@mein.io>
Wed, 19 Jan 2022 15:32:52 +0000 (16:32 +0100)
Filter the init script name parameter through fs.basename() to avoid
invoking paths outside of /etc/init.d/.

Reported-by: Graham R <gr348@cam.ac.uk>
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
modules/luci-base/luasrc/sys.lua

index aa00766fb87d8de247d9debc1d2319330e8c8761..af345a16d57291a0daffca53a58707180f3dfb9a 100644 (file)
@@ -566,6 +566,7 @@ function init.names()
 end
 
 function init.index(name)
+       name = fs.basename(name)
        if fs.access(init.dir..name) then
                return call("env -i sh -c 'source %s%s enabled; exit ${START:-255}' >/dev/null"
                        %{ init.dir, name })
@@ -573,6 +574,7 @@ function init.index(name)
 end
 
 local function init_action(action, name)
+       name = fs.basename(name)
        if fs.access(init.dir..name) then
                return call("env -i %s%s %s >/dev/null" %{ init.dir, name, action })
        end