-
Notifications
You must be signed in to change notification settings - Fork 369
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #594 from JOE1994/calamine
calamine: access unclaimed/uninitialized memory
- Loading branch information
Showing
1 changed file
with
19 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
```toml | ||
[advisory] | ||
id = "RUSTSEC-0000-0000" | ||
package = "calamine" | ||
date = "2021-01-06" | ||
url = "https://github.com/tafia/calamine/issues/199" | ||
categories = ["memory-corruption", "memory-exposure"] | ||
|
||
[versions] | ||
patched = [] | ||
``` | ||
|
||
# `Sectors::get` accesses unclaimed/uninitialized memory | ||
|
||
Affected versions of this crate arbitrarily calls `Vec::set_len` to increase length of a vector without claiming more memory for the vector. Affected versions of this crate | ||
also calls user-provided `Read` on the uninitialized memory of the vector that was | ||
extended with `Vec::set_len`. | ||
|
||
This can overwrite active entities in adjacent heap memory and seems to be a major security issue. Also, calling user-provided `Read` on uninitialized memory is defined as UB in Rust. |