A vulnerability exists in Pterodactyl Panel <= 1.6.6
that could allow a malicious attacker that compromises an API key to generate an authenticated user session that is not revoked when the API key is deleted, thus allowing the malicious user to remain logged in as the user the key belonged to.
It is important to note that a malicious user must first compromise an existing API key for a user to exploit this issue. It cannot be exploited by chance, and requires a coordinated attack against an individual account using a known API key.
This issue has been addressed in the v1.7.0
release of Pterodactyl Panel.
Those not wishing to upgrade may apply the change below:
diff --git a/app/Http/Middleware/Api/AuthenticateKey.php b/app/Http/Middleware/Api/AuthenticateKey.php
index eb25dac6..857bfab2 100644
--- a/app/Http/Middleware/Api/AuthenticateKey.php
+++ b/app/Http/Middleware/Api/AuthenticateKey.php
@@ -70,7 +70,7 @@ class AuthenticateKey
} else {
$model = $this->authenticateApiKey($request->bearerToken(), $keyType);
- $this->auth->guard()->loginUsingId($model->user_id);
+ $this->auth->guard()->onceUsingId($model->user_id);
}
If you have any questions or comments about this advisory please reach out to Tactical Fish#8008
on Discord or email dane@pterodactyl.io
.
{ "nvd_published_at": null, "cwe_ids": [ "CWE-613" ], "severity": "MODERATE", "github_reviewed": true, "github_reviewed_at": "2022-01-20T16:18:28Z" }