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
Description
Reviewed
Aug 5, 2021
Published to the GitHub Advisory Database
Aug 5, 2021
Withdrawn
Aug 24, 2021
Last updated
Jan 9, 2023
The
Deserialize
implementation forVecStorage
did not maintain the invariant that the number of elements must equalnrows * 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 ofDeserialize
forMatrixVec
.MatrixVec
was later renamed toVecStorage
in v0.16.13 (0f66403
) and continued to use the automatically derived implementation ofDeserialize
.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