GHSA-2gxj-qrp2-53jv

Suggest an improvement
Source
https://github.com/advisories/GHSA-2gxj-qrp2-53jv
Import Source
https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2022/01/GHSA-2gxj-qrp2-53jv/GHSA-2gxj-qrp2-53jv.json
JSON Data
https://api.osv.dev/v1/vulns/GHSA-2gxj-qrp2-53jv
Aliases
Published
2022-01-06T22:11:11Z
Modified
2023-11-08T04:07:22.377459Z
Severity
  • 9.8 (Critical) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:H/I:H/A:H CVSS Calculator
Summary
Incorrect reliance on Trait memory layout in mopa
Details

The mopa crate redefines the deprecated TraitObject struct from core::raw. This is done to then transmute a reference to a trait object (&dyn Trait for any trait Trait) into this struct and retrieve the data field for the purpose of downcasting. This is used to implement downcastrefunchecked(), in terms of which downcast_ref() is also implemented. Same goes for mutable reference downcasting and Box downcasting.

The Rust compiler explicitly reserves the right to change the memory layout of &dyn Trait for any trait Trait. The worst case scenario is that it swaps data and vtable, making an executable location breach and compromisation of ASLR possible, since reads from data would read vtable instead. Likewise, arbitrary code execution is also theoretically possible if reads of vtable generated by the compiler read data instead.

Database specific
{
    "nvd_published_at": "2021-12-27T00:15:00Z",
    "cwe_ids": [],
    "severity": "CRITICAL",
    "github_reviewed": true,
    "github_reviewed_at": "2022-01-06T14:22:40Z"
}
References

Affected packages

crates.io / mopa

Package

Affected ranges

Type
SEMVER
Events
Introduced
0Unknown introduced version / All previous versions are affected
Last affected
0.2.2