GHSA-8fv7-wq38-f5c9

Suggest an improvement
Source
https://github.com/advisories/GHSA-8fv7-wq38-f5c9
Import Source
https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2023/07/GHSA-8fv7-wq38-f5c9/GHSA-8fv7-wq38-f5c9.json
JSON Data
https://api.test.osv.dev/v1/vulns/GHSA-8fv7-wq38-f5c9
Aliases
Published
2023-07-28T15:34:36Z
Modified
2023-11-05T05:20:59.378977Z
Severity
  • 5.7 (Medium) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:L/UI:R/S:U/C:H/I:N/A:N CVSS Calculator
Summary
Cross-site scripting (XSS) from MIME type auto-detection of uploaded files
Details

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. Depending on your site, other JavaScript-powered attacks are possible.

Impact

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 serve the file with an incorrect MIME type of 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.

Patches

The problem has been patched in Kirby 3.5.8.3, Kirby 3.6.6.3, Kirby 3.7.5.2, Kirby 3.8.4.1 and Kirby 3.9.6. Please update to one of these or a later version to fix the vulnerability.

In all of the mentioned releases, we 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.

Credits

Thanks to Shankar Acharya (@5hank4r) for responsibly reporting the identified issue.

Database specific 
{
    "nvd_published_at": "2023-07-27T16:15:10Z",
    "cwe_ids": [
        "CWE-79"
    ],
    "severity": "MODERATE",
    "github_reviewed": true,
    "github_reviewed_at": "2023-07-28T15:34:36Z"
}
References

Affected packages

Packagist / getkirby/cms

Package

Name
getkirby/cms
Purl
pkg:composer/getkirby/cms

Affected ranges

Type
ECOSYSTEM
Events
Introduced
0Unknown introduced version / All previous versions are affected
Fixed
3.5.8.3

Affected versions

3.*

3.0.0
3.0.1-rc.1
3.0.1
3.0.2-rc.1
3.0.2
3.0.3-rc.1
3.0.3-rc.2
3.0.3-rc.3
3.0.3
3.1.0-rc.1
3.1.0
3.1.1
3.1.2-rc.1
3.1.2
3.1.3-rc.1
3.1.3
3.1.4-rc.1
3.1.4
3.2.0-rc.1
3.2.0-rc.2
3.2.0-rc.3
3.2.0-rc.4
3.2.0
3.2.1-rc.1
3.2.1
3.2.2
3.2.3-rc.1
3.2.3
3.2.4-rc.1
3.2.4
3.2.5-rc.1
3.2.5-rc.2
3.2.5
3.3.0-rc.1
3.3.0-rc.2
3.3.0-rc.3
3.3.0-rc.4
3.3.0-rc.5
3.3.0
3.3.1-rc.1
3.3.1
3.3.2-rc.1
3.3.2
3.3.3-rc.1
3.3.3
3.3.4-rc.1
3.3.4
3.3.5-rc.1
3.3.5
3.3.6
3.4.0-rc.1
3.4.0-rc.2
3.4.0-rc.3
3.4.0
3.4.1-rc.1
3.4.1
3.4.2
3.4.3-rc.1
3.4.3
3.4.4-rc.1
3.4.4
3.4.5
3.5.0-rc.1
3.5.0-rc.2
3.5.0-rc.3
3.5.0-rc.4
3.5.0-rc.5
3.5.0-rc.6
3.5.0-rc.7
3.5.0
3.5.1-rc.1
3.5.1
3.5.2-rc.1
3.5.2
3.5.3
3.5.3.1
3.5.4
3.5.5-rc.1
3.5.5
3.5.6-rc.1
3.5.6
3.5.7-rc.1
3.5.7
3.5.7.1
3.5.8
3.5.8.1
3.5.8.2

Packagist / getkirby/cms

Package

Name
getkirby/cms
Purl
pkg:composer/getkirby/cms

Affected ranges

Type
ECOSYSTEM
Events
Introduced
3.6.0
Fixed
3.6.6.3

Affected versions

3.*

3.6.0
3.6.1
3.6.1.1
3.6.2-rc.1
3.6.2-rc.2
3.6.2-rc.3
3.6.2
3.6.3-rc.1
3.6.3-rc.2
3.6.3
3.6.3.1
3.6.4-rc.1
3.6.4
3.6.5-rc.1
3.6.5
3.6.6-rc.1
3.6.6
3.6.6.1
3.6.6.2

Packagist / getkirby/cms

Package

Name
getkirby/cms
Purl
pkg:composer/getkirby/cms

Affected ranges

Type
ECOSYSTEM
Events
Introduced
3.7.0
Fixed
3.7.5.2

Affected versions

3.*

3.7.0
3.7.0.1
3.7.0.2
3.7.1-rc.1
3.7.1
3.7.2-rc.1
3.7.2
3.7.2.1
3.7.3-rc.1
3.7.3
3.7.4-rc.1
3.7.4
3.7.5
3.7.5.1

Packagist / getkirby/cms

Package

Name
getkirby/cms
Purl
pkg:composer/getkirby/cms

Affected ranges

Type
ECOSYSTEM
Events
Introduced
3.8.0
Fixed
3.8.4.1

Affected versions

3.*

3.8.0
3.8.1-rc.1
3.8.1
3.8.1.1
3.8.2-rc.1
3.8.2
3.8.3-rc.1
3.8.3-rc.2
3.8.3
3.8.4

Packagist / getkirby/cms

Package

Name
getkirby/cms
Purl
pkg:composer/getkirby/cms

Affected ranges

Type
ECOSYSTEM
Events
Introduced
3.9.0
Fixed
3.9.6

Affected versions

3.*

3.9.0
3.9.1-rc.1
3.9.1
3.9.2-rc.1
3.9.2
3.9.3-rc.1
3.9.3
3.9.4-rc.1
3.9.4
3.9.5-rc.1
3.9.5
3.9.6-rc.1