Skip to content

Improper Handling of Missing Values in kaml

Moderate severity GitHub Reviewed Published Sep 5, 2021 in charleskorn/kaml • Updated Feb 1, 2023

Package

maven com.charleskorn.kaml:kaml (Maven)

Affected versions

< 0.35.2

Patched versions

0.35.3

Description

Impact

Attackers that could provide arbitrary YAML input to an application that uses kaml could cause the application to endlessly loop while parsing the input. This could result in resource starvation and denial of service.

This only affects applications that use polymorphic serialization with the default tagged polymorphism style. Applications using the property polymorphism style are not affected.

YAML input for a polymorphic type that provided a tag but no value for the object would trigger the issue, for example:

!<x>

The following is a sample application that demonstrates this issue:

import com.charleskorn.kaml.Yaml
import kotlinx.serialization.SerialName
import kotlinx.serialization.Serializable

@Serializable
private sealed class K {
    @Serializable
    @SerialName("x")
    data class X(
        val property: String? = null,
    ) : K()
}

const val s = """
!<x>
"""

fun main() {
    println("Started.")
    val result = Yaml.default.decodeFromString(K.serializer(), s)
    println("Finished, result is $result")
}

On vulnerable versions of kaml, the decodeFromString() operation hangs and never returns.

Patches

Version 0.35.3 or later contain the fix for this issue.

References

@charleskorn charleskorn published to charleskorn/kaml Sep 5, 2021
Reviewed Sep 7, 2021
Published by the National Vulnerability Database Sep 7, 2021
Published to the GitHub Advisory Database Sep 7, 2021
Last updated Feb 1, 2023

Severity

Moderate
4.3
/ 10

CVSS base metrics

Attack vector
Network
Attack complexity
Low
Privileges required
Low
User interaction
None
Scope
Unchanged
Confidentiality
None
Integrity
None
Availability
Low
CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:U/C:N/I:N/A:L

CVE ID

CVE-2021-39194

GHSA ID

GHSA-fmm9-3gv8-58f4

Source code

Checking history
See something to contribute? Suggest improvements for this vulnerability.