rustsec-2021-0019
Vulnerability from osv_rustsec
Calls std::str::from_utf8_unchecked() without any checks
The function xcb::xproto::GetAtomNameReply::name() calls
std::str::from_utf8_unchecked() on the raw bytes that were received from the
X11 server without any validity checks. The X11 server only prevents interior
null bytes, but otherwise allows any X11 client to create an atom for arbitrary
bytes.
This issue is tracked here: https://github.com/rust-x-bindings/rust-xcb/issues/96
xcb::xproto::GetPropertyReply::value() allows arbitrary return types
The function xcb::xproto::GetPropertyReply::value() returns a slice of type
T where T is an unconstrained type parameter. The raw bytes received from
the X11 server are interpreted as the requested type.
The users of the xcb crate are advised to only call this function with the
intended types. These are u8, u16, and u32.
This issue is tracked here: https://github.com/rust-x-bindings/rust-xcb/issues/95
Out of bounds read in xcb::xproto::change_property()
xcb::xproto::change_property has (among others) the arguments format: u8 and
data: &[T]. The intended use is one of the following cases:
- format = 8 and T = u8
- format = 16 and T = u16
- format = 32 and T = u32
However, this constraint is not enforced. For example, it is possible to call
the function with format = 32 and T = u8. In this case, a read beyond the
end of the data slice is performed and the bytes are sent to the X11 server.
The users of the xcb crate are advised to only call this function with one of
the intended argument combinations.
This issue is tracked here: https://github.com/rust-x-bindings/rust-xcb/issues/94
'Safe' wrapper around std::mem::transmute()
The function xcb::base::cast_event() takes a reference to a
xcb::base::GenericEvent and returns a reference to an arbitrary type, as
requested by the caller (or found via type interference). The function is
implemented as a direct call to std::mem::transmute(). Since the return type
is not constrained, this allows transmution to an incorrect type or a type that
is larger than the X11 event that was passed in.
X11 events are mostly always 32 bytes large and this function works as intended.
Users are advised to only cast to the event structs provided by the xcb crate
(and hope for the best).
This issue is tracked here: https://github.com/rust-x-bindings/rust-xcb/issues/78
{
"affected": [
{
"database_specific": {
"categories": [
"memory-corruption",
"memory-exposure"
],
"cvss": null,
"informational": null
},
"ecosystem_specific": {
"affected_functions": null,
"affects": {
"arch": [],
"functions": [],
"os": []
}
},
"package": {
"ecosystem": "crates.io",
"name": "xcb",
"purl": "pkg:cargo/xcb"
},
"ranges": [
{
"events": [
{
"introduced": "0.0.0-0"
},
{
"fixed": "1.0.0"
}
],
"type": "SEMVER"
}
],
"versions": []
}
],
"aliases": [
"CVE-2021-26955",
"CVE-2021-26956",
"CVE-2021-26957",
"CVE-2021-26958",
"GHSA-2xpg-3hx4-fm9r",
"GHSA-3288-cwgw-ch86",
"GHSA-3cj3-jrrp-9rxf",
"GHSA-mp6r-fgw2-rxfx"
],
"database_specific": {
"license": "CC0-1.0"
},
"details": "## Calls `std::str::from_utf8_unchecked()` without any checks\n\nThe function `xcb::xproto::GetAtomNameReply::name()` calls\n`std::str::from_utf8_unchecked()` on the raw bytes that were received from the\nX11 server without any validity checks. The X11 server only prevents interior\nnull bytes, but otherwise allows any X11 client to create an atom for arbitrary\nbytes.\n\nThis issue is tracked here: https://github.com/rust-x-bindings/rust-xcb/issues/96\n\n## `xcb::xproto::GetPropertyReply::value()` allows arbitrary return types\n\nThe function `xcb::xproto::GetPropertyReply::value()` returns a slice of type\n`T` where `T` is an unconstrained type parameter. The raw bytes received from\nthe X11 server are interpreted as the requested type.\n\nThe users of the `xcb` crate are advised to only call this function with the\nintended types. These are `u8`, `u16`, and `u32`.\n\nThis issue is tracked here: https://github.com/rust-x-bindings/rust-xcb/issues/95\n\n## Out of bounds read in `xcb::xproto::change_property()`\n\n`xcb::xproto::change_property` has (among others) the arguments `format: u8` and\n`data: \u0026[T]`. The intended use is one of the following cases:\n- `format = 8` and `T = u8`\n- `format = 16` and `T = u16`\n- `format = 32` and `T = u32`\nHowever, this constraint is not enforced. For example, it is possible to call\nthe function with `format = 32` and `T = u8`. In this case, a read beyond the\nend of the `data` slice is performed and the bytes are sent to the X11 server.\n\nThe users of the `xcb` crate are advised to only call this function with one of\nthe intended argument combinations.\n\nThis issue is tracked here: https://github.com/rust-x-bindings/rust-xcb/issues/94\n\n## \u0027Safe\u0027 wrapper around `std::mem::transmute()`\n\nThe function `xcb::base::cast_event()` takes a reference to a\n`xcb::base::GenericEvent` and returns a reference to an arbitrary type, as\nrequested by the caller (or found via type interference). The function is\nimplemented as a direct call to `std::mem::transmute()`. Since the return type\nis not constrained, this allows transmution to an incorrect type or a type that\nis larger than the X11 event that was passed in.\n\nX11 events are mostly always 32 bytes large and this function works as intended.\n\nUsers are advised to only cast to the event structs provided by the `xcb` crate\n(and hope for the best).\n\nThis issue is tracked here: https://github.com/rust-x-bindings/rust-xcb/issues/78",
"id": "RUSTSEC-2021-0019",
"modified": "2023-06-13T13:10:24Z",
"published": "2021-02-04T12:00:00Z",
"references": [
{
"type": "PACKAGE",
"url": "https://crates.io/crates/xcb"
},
{
"type": "ADVISORY",
"url": "https://rustsec.org/advisories/RUSTSEC-2021-0019.html"
},
{
"type": "REPORT",
"url": "https://github.com/RustSec/advisory-db/issues/653"
},
{
"type": "REPORT",
"url": "https://github.com/rust-x-bindings/rust-xcb/issues/78"
},
{
"type": "REPORT",
"url": "https://github.com/rust-x-bindings/rust-xcb/issues/94"
},
{
"type": "REPORT",
"url": "https://github.com/rust-x-bindings/rust-xcb/issues/95"
},
{
"type": "REPORT",
"url": "https://github.com/rust-x-bindings/rust-xcb/issues/96"
}
],
"related": [],
"severity": [],
"summary": "Multiple soundness issues"
}
Sightings
| Author | Source | Type | Date |
|---|
Nomenclature
- Seen: The vulnerability was mentioned, discussed, or observed by the user.
- Confirmed: The vulnerability has been validated from an analyst's perspective.
- Published Proof of Concept: A public proof of concept is available for this vulnerability.
- Exploited: The vulnerability was observed as exploited by the user who reported the sighting.
- Patched: The vulnerability was observed as successfully patched by the user who reported the sighting.
- Not exploited: The vulnerability was not observed as exploited by the user who reported the sighting.
- Not confirmed: The user expressed doubt about the validity of the vulnerability.
- Not patched: The vulnerability was not observed as successfully patched by the user who reported the sighting.