Headline
CVE-2023-38491: Fix MIME detection vulnerability · getkirby/kirby@2f06ba1
Kirby is a content management system. A vulnerability in versions prior to 3.5.8.3, 3.6.6.3, 3.7.5.2, 3.8.4.1, and 3.9.6 affects all Kirby sites that might have potential attackers in the group of authenticated Panel users or that allow external visitors to upload an arbitrary file to the content folder. Kirby sites are not affected if they don’t allow file uploads for untrusted users or visitors or if the file extensions of uploaded files are limited to a fixed safe list. The attack requires user interaction by another user or visitor and cannot be automated.
An editor with write access to the Kirby Panel could upload a file with an unknown file extension like .xyz
that contains HTML code including harmful content like <script>
tags. The direct link to that file could be sent to other users or visitors of the site. If the victim opened that link in a browser where they are logged in to Kirby and the file had not been opened by anyone since the upload, Kirby would not be able to send the correct MIME content type, instead falling back to text/html
. The browser would then run the script, which could for example trigger requests to Kirby’s API with the permissions of the victim.
The issue was caused by the underlying Kirby\Http\Response::file()
method, which didn’t have an explicit fallback if the MIME type could not be determined from the file extension. If you use this method in site or plugin code, these uses may be affected by the same vulnerability.
The problem has been patched in Kirby 3.5.8.3, 3.6.6.3, 3.7.5.2, 3.8.4.1, and 3.9.6. In all of the mentioned releases, the maintainers have fixed the affected method to use a fallback MIME type of text/plain
and set the X-Content-Type-Options: nosniff
header if the MIME type of the file is unknown.
Expand Up
@@ -188,13 +188,41 @@ public function testJsonWithPrettyArray()
public function testFile()
{
$file = __DIR__ . '/fixtures/download.txt’;
$file = __DIR__ . '/fixtures/download.json’;
$response = Response::file($file);
$this->assertSame('application/json’, $response->type());
$this->assertSame(200, $response->code());
$this->assertSame('{"foo": "bar"}’, $response->body());
$response = Response::file($file, [
‘code’ => '201’,
‘headers’ => [
‘Pragma’ => ‘no-cache’
]
]);
$this->assertSame('application/json’, $response->type());
$this->assertSame(201, $response->code());
$this->assertSame('{"foo": "bar"}’, $response->body());
$this->assertSame([
‘Pragma’ => ‘no-cache’
], $response->headers());
}
public function testFileInvalid()
{
$file = __DIR__ . '/fixtures/download.xyz’;
$response = Response::file($file);
$this->assertSame('text/plain’, $response->type());
$this->assertSame(200, $response->code());
$this->assertSame('test’, $response->body());
$this->assertSame([
‘X-Content-Type-Options’ => 'nosniff’,
], $response->headers());
$response = Response::file($file, [
‘code’ => '201’,
Expand All
@@ -207,7 +235,8 @@ public function testFile()
$this->assertSame(201, $response->code());
$this->assertSame('test’, $response->body());
$this->assertSame([
‘Pragma’ => ‘no-cache’
‘Pragma’ => 'no-cache’,
‘X-Content-Type-Options’ => 'nosniff’,
], $response->headers());
}
Expand Down
Related news
### TL;DR This vulnerability affects all Kirby sites that might have potential attackers in the group of authenticated Panel users or that allow external visitors to upload an arbitrary file to the content folder. Your Kirby sites are *not* affected if they don't allow file uploads for untrusted users or visitors or if the file extensions of uploaded files are limited to a fixed safe list. The attack requires user interaction by another user or visitor and *cannot* be automated. ---- ### Introduction Cross-site scripting (XSS) is a type of vulnerability that allows to execute any kind of JavaScript code inside the Panel session of the same or other users. In the Panel, a harmful script can for example trigger requests to Kirby's API with the permissions of the victim. Such vulnerabilities are critical if you might have potential attackers in your group of authenticated Panel users. They can escalate their privileges if they get access to the Panel session of an admin user. Depen...