GHSA-x5mr-p6v4-wp93

Suggest an improvement
Source
https://github.com/advisories/GHSA-x5mr-p6v4-wp93
Import Source
https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2023/07/GHSA-x5mr-p6v4-wp93/GHSA-x5mr-p6v4-wp93.json
JSON Data
https://api.osv.dev/v1/vulns/GHSA-x5mr-p6v4-wp93
Aliases
Published
2023-07-28T15:35:47Z
Modified
2024-02-16T08:11:42.659513Z
Severity
  • 7.1 (High) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:U/C:N/I:H/A:L CVSS Calculator
Summary
Field injection in the KirbyData text storage handler
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 update a Kirby content file (e.g. via a contact or comment form).

Your Kirby sites are not affected if they don't allow write access for untrusted users or visitors.

Introduction

A field injection in a content storage implementation is a type of vulnerability that allows attackers with content write access to overwrite content fields that the site developer didn't intend to be modified.

In a Kirby site this can be used to alter site content, break site behavior or inject malicious data or code. The exact security risk depends on the field type and usage.

Impact

Kirby stores content of the site, of pages, files and users in text files by default. The text files use Kirby's KirbyData format where each field is separated by newlines and a line with four dashes (----).

When reading a KirbyData file, the affected code first removed the Unicode BOM sequence from the file contents and afterwards split the content into fields by the field separator.

When writing to a KirbyData file, field separators in field data are escaped to prevent user input from interfering with the field structure. However this escaping could be tricked by including a Unicode BOM sequence in a field separator (e.g. --\xEF\xBB\xBF--). When writing, this was not detected as a separator, but during the read process the BOM was removed, turning the malicious line into a valid separator. This could be abused by attackers to inject other field data into content files.

Because each field can only be defined once per content file, this vulnerability only affects fields in the content file that were defined above the vulnerable user-writable field or not at all. Fields that are defined below the vulnerable field override the injected field content and were therefore already protected.

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 code to only remove the Unicode BOM sequence at the beginning of the file. This fixes this vulnerability both for newly written as well as for existing content files.

Credits

Thanks to Patrick Falb (@dapatrese) at FORMER 03 for responsibly reporting the identified issue.

Database specific 
{
    "nvd_published_at": "2023-07-27T15:15:11Z",
    "cwe_ids": [
        "CWE-140",
        "CWE-863"
    ],
    "severity": "HIGH",
    "github_reviewed": true,
    "github_reviewed_at": "2023-07-28T15:35:47Z"
}
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