CVE-2023-46453 is a remote authentication bypass vulnerability in the web
interface of GLiNet routers running firmware versions 4.x and up. The
vulnerability allows an attacker to bypass authentication and gain access
to the router's web interface.
The vulnerability is caused by a lack of proper authentication checks in
/usr/sbin/gl-ngx-session file. The file is responsible for authenticating
users to the web interface. The authentication is in different stages.
Stage 1:
During the first stage the user send a request to the challenge rcp
endpoint. The endpoint returns a random nonce value used later in the
authentication process.
Stage 2:
During the second stage the user sends a request to the login rcp endpoint
with the username and the encrypted password. The encrypted password is
calculated by the following formula:
md5(username + crypt(password) + nonce)
The crypt function is the standard unix crypt function.
The vulnerability lies in the fact that the username is not sanitized
properly before being passed to the login_test function in the lua script.
------------------------------------------------------------------------
local function login_test(username, hash)
if not username or username == "" then return false end
for l in io.lines("/etc/shadow") do
local pw = l:match('^' .. username .. ':([^:]+)')
if pw then
for nonce in pairs(nonces) do
if utils.md5(table.concat({username, pw, nonce}, ":")) ==
hash then
nonces[nonce] = nil
nonce_cnt = nonce_cnt - 1
return true
end
end
return false
end
end
return false
end
------------------------------------------------------------------------
This script check the username against the /etc/shadow file. If the username
is found in the file the script will extract the password hash and compare
it to the hash sent by the user. If the hashes match the user is
authenticated.
The issue is that the username is not sanitized properly before being
concatenated with the regex. This allows an attacker to inject a regex into
the username field and modify the final behavior of the regex.
for instance, the following username will match the userid of the root user:
root:[^:]+:[^:]+ will become root:[^:]+:[^:]+:([^:]+)
This will match the "root:" string and then any character until the next ":"
character. This will cause the script skip the password and return the
user id instead.
Since the user id of the root user is always 0, the script will always
return:
md5("root:[^:]+:[^:]+" + "0" + nonce)
Since this value is always the same, the attacker can simply send the known
hash value to the login rcp endpoint and gain access to the web interface.
Anyway this approach won't work as expected since later in the code inside
the
this check appear:
------------------------------------------------------------------------
local aclgroup = db.get_acl_by_username(username)
The username which is now our custom regex will be passed to the
get_acl_by_username
function. This function will check the username against a database and
return the aclgroup associated with the username.
If the username is not found in the database the function will return nil,
thus causing attack to fail.
By checking the code we can see that the get_acl_by_username function is
actually appending our raw string to a query and then executing it.
This means that we can inject a sql query into the username field and
make it return a valid aclgroup.
------------------------------------------------------------------------
M.get_acl_by_username = function(username)
if username == "root" then return "root" end
local db = sqlite3.open(DB)
local sql = string.format("SELECT acl FROM account WHERE username =
'%s'", username)
local aclgroup = ""
for a in db:rows(sql) do
aclgroup = a[1]
end
db:close()
return aclgroup
end
------------------------------------------------------------------------
Using this payload we were able to craft a username which is both a valid
regex and a valid sql query: