GHSA-h352-g5vw-3926

Source

https://github.com/advisories/GHSA-h352-g5vw-3926

Import Source

https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2021/11/GHSA-h352-g5vw-3926/GHSA-h352-g5vw-3926.json

JSON Data

https://api.osv.dev/v1/vulns/GHSA-h352-g5vw-3926

Aliases

Published

2021-11-16T17:26:47Z

Modified

2023-11-08T04:07:11.140379Z

Severity

7.5 (High) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:H/A:N CVSS Calculator

Summary

Improper Input Validation in fruity

Details

Methods of NSString for conversion to a string may return a partial result. Since they call CStr::from_ptr on a pointer to the string buffer, the string is terminated at the first null byte, which might not be the end of the string.

In addition to the vulnerable functions listed for this issue, the implementations of Display, PartialEq, PartialOrd, and ToString for NSString are also affected, since they call those functions.

Since NSString is commonly used as the type for paths by the Foundation framework, null byte truncation might allow for easily bypassing file extension checks. For example, if a file name is provided by a user and validated to have one of a specific set of extensions, with validation taking place before truncation, an attacker can add an accepted extension after a null byte (e.g., file.exe\0.txt). After truncation, the file name used by the application would be file.exe.

It would be better to generate unique names for files, instead of using user-provided names, but not all applications take this approach.

Database specific

{
    "nvd_published_at": "2021-11-15T05:15:00Z",
    "cwe_ids": [
        "CWE-20"
    ],
    "severity": "HIGH",
    "github_reviewed": true,
    "github_reviewed_at": "2021-11-16T15:32:42Z"
}

References

Affected packages

crates.io / fruity

Package

Name: fruity; View open source insights on deps.dev
Purl: pkg:cargo/fruity

Affected ranges

Type: SEMVER
Events: Introduced

0.1.0

Fixed

0.3.0

Ecosystem specific

{
    "affected_functions": [
        "fruity::foundation::NSString::to_str",
        "fruity::foundation::NSString::to_str_with_nul",
        "fruity::foundation::NSString::to_string",
        "fruity::foundation::NSString::to_string_with_nul"
    ]
}