Skip to content

VecStorage Deserialize Allows Violation of Length Invariant

Moderate severity GitHub Reviewed Published Aug 5, 2021 to the GitHub Advisory Database • Updated Jan 9, 2023
Withdrawn This advisory was withdrawn on Aug 24, 2021

Package

cargo nalgebra (Rust)

Affected versions

>= 0.11.0, < 0.27.0

Patched versions

0.27.0

Description

The Deserialize implementation for VecStorage did not maintain the invariant that the number of elements must equal nrows * ncols. Deserialization of specially crafted inputs could allow memory access beyond allocation of the vector.

This flaw was introduced in v0.11.0 (086e6e) due to the addition of an automatically derived implementation of Deserialize for MatrixVec. MatrixVec was later renamed to VecStorage in v0.16.13 (0f66403) and continued to use the automatically derived implementation of Deserialize.

This flaw was corrected in commit 5bff536 by returning an error during deserialization if the number of elements does not exactly match the expected size.

References

Reviewed Aug 5, 2021
Published to the GitHub Advisory Database Aug 5, 2021
Withdrawn Aug 24, 2021
Last updated Jan 9, 2023

Severity

Moderate

Weaknesses

No CWEs

CVE ID

No known CVE

GHSA ID

GHSA-h3mf-4fwp-59c7

Source code

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