Ecosyste.ms: Advisories

Security Advisories: GSA_kwCzR0hTQS13cnc3LTg5anAtOHE4Z84ABCrZ

Unsoundness in `Iterator` and `DoubleEndedIterator` impls for `glib::VariantStrIter`

The VariantStrIter::impl_get function (called internally by implementations of the Iterator and DoubleEndedIterator traits for this type) was unsound, resulting in undefined behaviour.

An immutable reference &p to a *mut libc::c_char pointer initialized to NULL was passed as an argument to a C function that that mutates the pointer behind &p in-place (i.e. as an out-argument), which was unsound. After changes in recent versions of the Rust compiler, these unsound writes through &p now seem to be completely disregarded when building the glib crate with optimizations.

This subsequently caused all calls of VariantStrIter::impl_get to violate the safety requirements of the std::ffi::CStr::from_ptr function - which requires its argument to be a valid pointer to a C-style string - resulting in crashes due to NULL pointer dereferences.

This was fixed by passing the out-argument pointer explitly as &mut p instead of &p.

This issue has been present since this code was initially added in glib v0.15.0. The mismatch in mutability was likely missed (and not raised as an error by the compiler) because the C function wrapped by VariantStrIter::impl_get is variadic (glib_sys::g_variant_get_child), and the pointer in question is one of the variadic arguments.

Permalink: https://github.com/advisories/GHSA-wrw7-89jp-8q8g
JSON: https://advisories.ecosyste.ms/api/v1/advisories/GSA_kwCzR0hTQS13cnc3LTg5anAtOHE4Z84ABCrZ
Source: GitHub Advisory Database
Origin: Unspecified
Severity: Moderate
Classification: General
Published: 29 days ago
Updated: 29 days ago

Identifiers: GHSA-wrw7-89jp-8q8g
References:

• https://github.com/gtk-rs/gtk-rs-core/pull/1343
• https://rustsec.org/advisories/RUSTSEC-2024-0429.html
• https://github.com/advisories/GHSA-wrw7-89jp-8q8g

Repository: https://github.com/gtk-rs/gtk-rs-core
Blast Radius: 0.0

Affected Packages